Skip to content

Handle scanner POSTs to catch-all routes#233

Closed
kentcdodds wants to merge 1 commit into
mainfrom
cleanup-sentry-internal-router-error-noise
Closed

Handle scanner POSTs to catch-all routes#233
kentcdodds wants to merge 1 commit into
mainfrom
cleanup-sentry-internal-router-error-noise

Conversation

@kentcdodds

@kentcdodds kentcdodds commented May 17, 2026

Copy link
Copy Markdown
Member

Summary

  • Patch the packaged workshop app server build after install so the root catch-all route has an action that throws a normal 404 response for POST submissions.
  • Add a Playwright regression test for scanner-style POSTs to /_middleware.
  • Keep the filter narrow by preserving existing route behavior and avoiding a broad Sentry getInternalRouterError suppression.

Sentry

  • epicshop issue EPICSHOP-F0 / 7447885629
  • Examples included POST probes for /_middleware, /.action, /RSC/...txt, /_next, /__rsc, /_rsc, and /_next/image.

Validation

  • npm test -- catch-all-posts.spec.js (from epicshop/)
  • npm run lint
  • npm run typecheck
Open in Web Open in Cursor 

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