Skip to content

Handle scanner POSTs to catch-all routes#254

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

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

Conversation

@kentcdodds
Copy link
Copy Markdown
Member

@kentcdodds kentcdodds commented May 17, 2026

Summary

  • add an install-time patch for @epic-web/workshop-app so the routes/$ catch-all has a normal 404 action for unsupported method requests
  • keep the fix scoped to the catch-all/splat route instead of suppressing all getInternalRouterError events
  • add a focused node test covering the patch and idempotency

Sentry

  • epicshop EPICSHOP-F0 / 7447885629
  • Example noisy request: POST *splat /connectors/resource/index.php

Validation

  • npm --prefix epicshop run postinstall
  • npm --prefix epicshop run test:patch
  • npm run typecheck
  • npm run lint
  • Smoke: POST /connectors/resource/index.php returned HTTP/1.1 404 Not Found with no server error log
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