Skip to content

Conversation

@mcollina
Copy link
Member

@mcollina mcollina commented Jan 3, 2026

URL-encoded paths could bypass middleware (e.g., /%61dmin would bypass middleware registered on /admin). This uses FindMyWay.sanitizeUrlPath() to decode URLs before Express matches middleware, consistent with the fix in fastify/middie#245.

Checklist

URL-encoded paths could bypass middleware (e.g., /%61dmin would bypass
middleware registered on /admin). This uses FindMyWay.sanitizeUrlPath()
to decode URLs before Express matches middleware, consistent with the
fix in fastify/middie#245.
@mcollina mcollina requested a review from Eomm January 3, 2026 17:05
@mcollina mcollina requested a review from gurgunday January 5, 2026 18:20
@Eomm Eomm merged commit dc02a3f into main Jan 5, 2026
29 of 31 checks passed
@Eomm Eomm deleted the fix/decode-paths-before-matching branch January 5, 2026 18:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants