Skip to content

fix(react): Parameterize React Router transactions with encoded params - #25639

Open
seongwon030 wants to merge 1 commit into
getsentry:developfrom
seongwon030:fix/react-router-encoded-pathname
Open

seongwon030 wants to merge 1 commit into
getsentry:developfrom
seongwon030:fix/react-router-encoded-pathname

Conversation

@seongwon030

Copy link
Copy Markdown

Closes #25638

  • If you've added code that should be tested, please add tests.
  • Ensure your code lints and the test suite passes (pnpm run lint) & (pnpm run test).
  • Link an issue if there is one related to your pull request. If no issue is linked, one will be auto-generated and linked.

location.pathname is percent-encoded, but React Router decodes the pathname before matching (decodePath), so RouteMatch.pathname and pathnameBase are decoded. The React Router instrumentation compared the two with plain string equality, so a route like /clubs/:clubId visited as /clubs/%ED%95%9C%EA%B8%80 never matched and the transaction fell back to the raw URL with source url. Every non-ASCII parameter value became its own transaction name. Descendant routes (<Routes> nested under a /* route) produced the raw URL with source route.

This adds decodePathname, the same segment-wise algorithm as React Router's decodePath, and applies it to both sides of the comparison in getNormalizedName and of the pathnameBase stripping for descendant routes. Decoding both sides keeps percent-encoded basenames and older 6.x routers working: before 6.22.1 matchRoutes matched against the raw pathname or used decodeURI, so there the branch pathname is the side that is still encoded. A value that still looks percent-encoded after one decode (a literal %41) is decoded once more; such values were not parameterized before either.

Validation (in packages/react):

  • vitest run on the React Router suites (react-router, reactrouterv6, reactrouter-descendant-routes, reactrouter-cross-usage, reactrouter-compat-utils/*): 276 tests passed, including the new ones
  • The new decodePathname, getNormalizedName, rebuildRoutePathFromAllRoutes, v7 pageload/navigation and v6 descendant-route tests fail without the fix
  • pnpm run lint clean; oxfmt --check passed on the changed files
  • Browser integration tests (Playwright) were not run

`location.pathname` is percent-encoded, but React Router decodes the pathname
before matching, so `RouteMatch.pathname` and `pathnameBase` are decoded. The
instrumentation compared the two with plain string equality, so a route like
`/clubs/:clubId` visited as `/clubs/%ED%95%9C%EA%B8%80` never matched and the
transaction fell back to the raw URL with source `url`.

Decode both sides, segment by segment like React Router's `decodePath`, before
comparing in `getNormalizedName` and before stripping `pathnameBase` for
descendant routes.
@seongwon030
seongwon030 requested a review from a team as a code owner October 22, 2026 03:18
@seongwon030
seongwon030 requested review from chargome and s1gr1d and removed request for a team October 22, 2026 03:18
@github-actions github-actions Bot added the external PR from an external contributor label Oct 22, 2026

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

external PR from an external contributor

Projects

None yet

Development

Successfully merging this pull request may close these issues.

React Router transactions are not parameterized when the URL has a percent-encoded parameter

1 participant