Skip to content

Handle noisy scanner posts before workshop router#6

Closed
kentcdodds wants to merge 2 commits into
mainfrom
cleanup-sentry-internal-router-error-noise
Closed

Handle noisy scanner posts before workshop router#6
kentcdodds wants to merge 2 commits into
mainfrom
cleanup-sentry-internal-router-error-noise

Conversation

@kentcdodds
Copy link
Copy Markdown
Member

@kentcdodds kentcdodds commented May 17, 2026

Summary

  • Patch the packaged workshop app server during epicshop install so known scanner mutation probes are answered with a normal 404 before React Router reaches routes/$.
  • Keep normal GET/HEAD/OPTIONS and all non-scanner app routes flowing through the existing router.
  • Add a focused unit test for the postinstall patcher.

Sentry

  • References epicshop issue EPICSHOP-F0 / 7447885629 (getInternalRouterError noise from POSTs to catch-all/splat scanner paths like /__rsc, /_next/image, /_rsc, /.action, /__nextjs_action, /_middleware, and /RSC/...txt).

Validation

  • npm run test:patch-workshop-app --prefix ./epicshop
  • npm ci (confirmed postinstall applies the workshop app server patch)
  • npm run typecheck
  • npm run lint
  • node ./epicshop/test.js ..s
  • Smoke test against npm start: scanner POSTs to /, /__rsc, /_next/image, /_rsc, /_next, /.action, /__nextjs_action, /_middleware, and /RSC/probe.txt returned normal 404 responses; POST /progress still routed normally.
Open in Web Open in Cursor 

cursoragent and others added 2 commits May 17, 2026 14:15
Co-authored-by: Kent C. Dodds <me+github@kentcdodds.com>
Co-authored-by: Kent C. Dodds <me+github@kentcdodds.com>
@kody-bot kody-bot closed this May 17, 2026
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