GHSA-F842-PHM9-P4V4
Vulnerability from github – Published: 2026-03-19 12:44 – Updated: 2026-03-25 20:48Details
A Path Traversal and Access Control Bypass vulnerability was discovered in the salvo-proxy component of the Salvo Rust framework (v0.89.2). The vulnerability allows an unauthenticated external attacker to bypass proxy routing constraints and access unintended backend paths (e.g., protected endpoints or administrative dashboards). This issue stems from the encode_url_path function, which fails to normalize "../" sequences and inadvertently forwards them verbatim to the upstream server by not re-encoding the "." character.
Technical Details
If someone tries to attack by sending a special code like %2e%2e, Salvo changes it back to ../ when it first checks the path. The proxy then gets this plain ../ value. When making the new URL to send forward, the encode_url_path function tries to change the path again, but its normal settings do not include the . (dot) character. Because of this, the proxy puts ../ straight into the new URL and sends a request like GET /api/../admin HTTP/1.1 to the backend server.
// crates/proxy/src/lib.rs (Lines 100-105)
pub(crate) fn encode_url_path(path: &str) -> String {
path.split('/')
.map(|s| utf8_percent_encode(s, PATH_ENCODE_SET).to_string())
.collect::<Vec<_>>()
.join("/")
}
PoC
1 - Setup an Nginx Backend Server for example 2 - Start Salvo Proxy Gateway in other port routing to /api/ 3 - Run the curl to test the bypass:
curl -s http://127.0.0.1:8080/gateway/api/%2e%2e%2fadmin/index.html
Impact
If attackers take advantage of this problem, they can get past API Gateway security checks and route limits without logging in. This could accidentally make internal services, admin pages, or folders visible.
The attack works because the special path is sent as-is to the backend, which often happens in systems that follow standard web address rules. Attackers might also use different ways of writing URLs or add extra parts to the web address to get past simple security checks that only look for exact ../ patterns.
Remediation
Instead of changing the text of the path manually, the proxy should use a standard way to clean up the path according to RFC 3986 before adding it to the main URL. It is better to use a trusted tool like the URL crate to join paths, or to block any path parts with “..” after decoding them. But a custom implementation maybe looks like:
pub(crate) fn encode_url_path(path: &str) -> String {
let normalized = normalize_path(path);
normalized.split('/')
.map(|s| utf8_percent_encode(s, PATH_ENCODE_SET).to_string())
.collect::<Vec<_>>()
.join("/")
}
fn normalize_path(path: &str) -> String {
let mut stack = Vec::new();
for part in path.split('/') {
match part {
"" | "." => (),
".." => { let _ = stack.pop(); },
_ => stack.push(part),
}
}
stack.join("/")
}
Vulnerable code introduced in: https://github.com/salvo-rs/salvo/commit/7bac30e6960355c58e358e402072d4a3e5c4e1bb#diff-e319bf7afcb577f7e9f4fb767005072f6335d23f306dd52e8c94f3d222610d02R20
Author: Tomas Illuminati
{
"affected": [
{
"database_specific": {
"last_known_affected_version_range": "\u003c= 0.89.2"
},
"package": {
"ecosystem": "crates.io",
"name": "salvo"
},
"ranges": [
{
"events": [
{
"introduced": "0.39.0"
},
{
"fixed": "0.89.3"
}
],
"type": "ECOSYSTEM"
}
]
}
],
"aliases": [
"CVE-2026-33242"
],
"database_specific": {
"cwe_ids": [
"CWE-22"
],
"github_reviewed": true,
"github_reviewed_at": "2026-03-19T12:44:28Z",
"nvd_published_at": "2026-03-24T00:16:29Z",
"severity": "HIGH"
},
"details": "### Details\n\nA Path Traversal and Access Control Bypass vulnerability was discovered in the salvo-proxy component of the Salvo Rust framework (v0.89.2). The vulnerability allows an unauthenticated external attacker to bypass proxy routing constraints and access unintended backend paths (e.g., protected endpoints or administrative dashboards). This issue stems from the encode_url_path function, which fails to normalize \"../\" sequences and inadvertently forwards them verbatim to the upstream server by not re-encoding the \".\" character.\n\n---\n\n### Technical Details\n\nIf someone tries to attack by sending a special code like `%2e%2e`, Salvo changes it back to `../` when it first checks the path. The proxy then gets this plain `../` value. When making the new URL to send forward, the `encode_url_path` function tries to change the path again, but its normal settings do not include the `.` (dot) character. Because of this, the proxy puts `../` straight into the new URL and sends a request like `GET /api/../admin HTTP/1.1` to the backend server.\n\n```rust\n// crates/proxy/src/lib.rs (Lines 100-105)\n\npub(crate) fn encode_url_path(path: \u0026str) -\u003e String {\n path.split(\u0027/\u0027)\n .map(|s| utf8_percent_encode(s, PATH_ENCODE_SET).to_string())\n .collect::\u003cVec\u003c_\u003e\u003e()\n .join(\"/\")\n}\n```\n\n---\n\n### PoC\n\n1 - Setup an Nginx Backend Server for example\n2 - Start Salvo Proxy Gateway in other port routing to /api/\n3 - Run the curl to test the bypass:\n\n```bash\ncurl -s http://127.0.0.1:8080/gateway/api/%2e%2e%2fadmin/index.html\n```\n\n---\n\n### Impact\n\nIf attackers take advantage of this problem, they can get past API Gateway security checks and route limits without logging in. This could accidentally make internal services, admin pages, or folders visible.\n\nThe attack works because the special path is sent as-is to the backend, which often happens in systems that follow standard web address rules. Attackers might also use different ways of writing URLs or add extra parts to the web address to get past simple security checks that only look for exact `../` patterns.\n\n---\n\n### Remediation\n\nInstead of changing the text of the path manually, the proxy should use a standard way to clean up the path according to RFC 3986 before adding it to the main URL. It is better to use a trusted tool like the URL crate to join paths, or to block any path parts with \u201c..\u201d after decoding them. But a custom implementation maybe looks like:\n\n```rust\npub(crate) fn encode_url_path(path: \u0026str) -\u003e String {\n let normalized = normalize_path(path);\n normalized.split(\u0027/\u0027)\n .map(|s| utf8_percent_encode(s, PATH_ENCODE_SET).to_string())\n .collect::\u003cVec\u003c_\u003e\u003e()\n .join(\"/\")\n}\n\nfn normalize_path(path: \u0026str) -\u003e String {\n let mut stack = Vec::new();\n for part in path.split(\u0027/\u0027) {\n match part {\n \"\" | \".\" =\u003e (), \n \"..\" =\u003e { let _ = stack.pop(); },\n _ =\u003e stack.push(part),\n }\n }\n stack.join(\"/\")\n}\n```\n---\n\n**Vulnerable code introduced in:**\nhttps://github.com/salvo-rs/salvo/commit/7bac30e6960355c58e358e402072d4a3e5c4e1bb#diff-e319bf7afcb577f7e9f4fb767005072f6335d23f306dd52e8c94f3d222610d02R20\n\n---\n\n**Author**: Tomas Illuminati",
"id": "GHSA-f842-phm9-p4v4",
"modified": "2026-03-25T20:48:34Z",
"published": "2026-03-19T12:44:28Z",
"references": [
{
"type": "WEB",
"url": "https://github.com/salvo-rs/salvo/security/advisories/GHSA-f842-phm9-p4v4"
},
{
"type": "ADVISORY",
"url": "https://nvd.nist.gov/vuln/detail/CVE-2026-33242"
},
{
"type": "WEB",
"url": "https://github.com/salvo-rs/salvo/commit/7bac30e6960355c58e358e402072d4a3e5c4e1bb#diff-e319bf7afcb577f7e9f4fb767005072f6335d23f306dd52e8c94f3d222610d02R20"
},
{
"type": "PACKAGE",
"url": "https://github.com/salvo-rs/salvo"
},
{
"type": "WEB",
"url": "https://github.com/salvo-rs/salvo/releases/tag/v0.89.3"
}
],
"schema_version": "1.4.0",
"severity": [
{
"score": "CVSS:3.1/AV:N/AC:L/PR:N/UI:N/S:U/C:H/I:N/A:N",
"type": "CVSS_V3"
}
],
"summary": "Salvo has a Path Traversal in salvo-proxy::encode_url_path allows API Gateway Bypass"
}
Sightings
| Author | Source | Type | Date |
|---|
Nomenclature
- Seen: The vulnerability was mentioned, discussed, or observed by the user.
- Confirmed: The vulnerability has been validated from an analyst's perspective.
- Published Proof of Concept: A public proof of concept is available for this vulnerability.
- Exploited: The vulnerability was observed as exploited by the user who reported the sighting.
- Patched: The vulnerability was observed as successfully patched by the user who reported the sighting.
- Not exploited: The vulnerability was not observed as exploited by the user who reported the sighting.
- Not confirmed: The user expressed doubt about the validity of the vulnerability.
- Not patched: The vulnerability was not observed as successfully patched by the user who reported the sighting.