RestApiTool path parameters with '.' / '..' still reach the wire after quote(safe="")
@sanketpatil06 đang làm issue này rồi.
Từ ngày 9/9/2026.
Đánh giá
Issue này chưa được đánh giá.
Mô tả
🔴 Required Information
Describe the Bug:
RestApiTool (google.adk.tools.openapi_tool) percent-encodes path parameter values with urllib.parse.quote(value, safe="") and documents that this prevents a model-supplied value from redirecting the request onto an undeclared path on the same host.
That guarantee does not hold for RFC 3986 dot-segments. quote() never encodes . (it is unreserved), so a value containing .. is sent with literal dot-dot segments joined by encoded slashes. Backends/gateways that decode %2F and then merge dot-segments can dispatch the request — with the tool's configured credentials — to a path the OpenAPI spec never declared.
>>> from urllib.parse import quote
>>> quote("../../admin/secret", safe="")
'..%2F..%2Fadmin%2Fsecret'
This is a hardening follow-up to the merged encoding fix (25f53bd). Google VRP issue 557701521 was closed as Infeasible (not tracked as a security bug) with a request to file this publicly.
Steps to Reproduce:
- Use an OpenAPI spec that declares only
GET /files/{name}against a host that also serves an undeclared route such asGET /admin/secret. - Call the generated tool with
args={"name": "../../admin/secret"}. - Observe the outgoing request line
GET /files/..%2F..%2Fadmin%2Fsecret. - On a backend that decodes
%2Fthen merges dot-segments (e.g. a Go router overpath.Clean(r.URL.Path), nginx as a gateway), the undeclared/admin/secretbody is returned as the tool result.
Expected Behavior:
Path parameters whose /- or \-separated segments are . or .. are rejected before any HTTP request is sent. quote(safe="") continues to encode /, ?, and #. Slash-containing IDs such as foo/bar remain encoded as foo%2Fbar.
Observed Behavior:
The client emits GET /files/..%2F..%2Fadmin%2Fsecret. httpx correctly treats %2F as data, so the dot-dot sequences reach the backend unchanged.
Environment Details:
- ADK Library Version (pip show google-adk): 2.8.0 (
a119dd7751082dbbd9a65f71e359abdc2be659cc) - Desktop OS: macOS
- Python Version (python -V): 3.12 / 3.13
Model Information:
- Are you using LiteLLM: N/A (library-level RestApiTool)
- Which model is being used: N/A
🟡 Optional Information
Regression:
Present in any release that contains the quote(safe="") path-param encoding fix.
Minimal Reproduction Code:
from urllib.parse import quote
print(quote("../../admin/secret", safe="")) # '..%2F..%2Fadmin%2Fsecret'
Additional Context:
Backend behavior for GET /files/..%2F..%2Fadmin%2Fsecret:
- Hand-rolled Go routers that clean the decoded path (
path.Clean(r.URL.Path)): request is dispatched to/admin/secret. - Legacy Go ServeMux (pre-1.22 /
GODEBUG=httpmuxgo121=1): 301 to/admin/secret. ADK does not follow redirects; a following intermediary would complete the access. - Modern Go ServeMux (>= 1.22), FastAPI/Starlette direct, Envoy defaults: not affected (
%2Fstays one segment). - nginx as gateway: decode-and-merge by documented analysis.
Same host/port only (origin is fixed by the spec). Suggested client-side fix: reject . / .. as path segments, then keep quote(safe="").
- Ngôn ngữ chính
- Python
- Star
- 21.6k
- Fork
- 4k
- Merge trung bình
- 13 giờ 49 phút
- Pull request đã merge (30 ngày)
- 10
Hướng dẫn đóng góp
Bắt đầu từ đâu
- Đọc hết issue, rồi đọc hướng dẫn đóng góp của dự án.
- Bình luận trên issue rằng bạn sẽ nhận — tránh hai người làm cùng một việc.
- Fork repository và làm thay đổi trên một nhánh.
- Mở pull request có tham chiếu số hiệu của issue.
Issue khác của google/adk-python
-
mcp
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 75/100
google/adk-python#7217 · 3 bình luận · 1 người được giao ·
-
tools
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 78/100
google/adk-python#7206 · 1 bình luận · 1 người được giao ·
-
request clarification tools
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 86/100
google/adk-python#7205 · 2 bình luận · 1 người được giao ·
-
mcp
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 78/100
google/adk-python#7196 · 1 bình luận · 1 người được giao ·
-
eval request clarification
Độ khó 1/5 1-3 giờ Mức phù hợp với người mới 86/100
google/adk-python#7146 · 2 bình luận · 1 người được giao ·
Tất cả issue của google/adk-python
Issue tương tự
-
essnmx good first issue
Độ khó 1/5 Dưới một giờ Mức phù hợp với người mới 95/100
-
[Feature] 奇物选择添加优先级 Đang mở
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 65/100
syfoud/Simulated_Scepter#174 ·
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 75/100
Giskard-AI/giskard-oss#2840 · 1 bình luận ·
-
A claim comment carrying the issue number is silently declined while the workflow reports success Đang mởarea: repo bug perceived difficulty: 2
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 70/100
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 75/100
yeti-platform/yeti#1380 ·