Skip to content

fix(server): handle bad urls#6098

Merged
trpc-bot merged 22 commits into
nextfrom
fix-bad-mw
Oct 11, 2024
Merged

fix(server): handle bad urls#6098
trpc-bot merged 22 commits into
nextfrom
fix-bad-mw

Conversation

@KATT

@KATT KATT commented Oct 10, 2024

Copy link
Copy Markdown
Member

Closes #6094

🎯 Changes

Makes all base HTTP handlers sync rather than async and handle exceptions

  • Add regression test
  • Fix regression test

@KATT KATT requested a review from a team as a code owner October 10, 2024 08:34
@railway-app

railway-app Bot commented Oct 10, 2024

Copy link
Copy Markdown

🚅 Previously deployed to Railway in the trpc-sse-and-websockets project. Environment has been deleted.

@vercel

vercel Bot commented Oct 10, 2024

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
next-prisma-starter ✅ Ready (Inspect) Visit Preview Oct 11, 2024 0:38am
og-image ✅ Ready (Inspect) Visit Preview 💬 Add feedback Oct 11, 2024 0:38am
1 Skipped Deployment
Name Status Preview Comments Updated (UTC)
www ⬜️ Ignored (Inspect) Visit Preview Oct 11, 2024 0:38am

KATT added 2 commits October 11, 2024 01:47
This reverts commit c4926b0.
trpc-bot
trpc-bot previously approved these changes Oct 11, 2024
trpc-bot
trpc-bot previously approved these changes Oct 11, 2024
@github-actions

Copy link
Copy Markdown
Contributor

This pull request has been locked because we are very unlikely to see comments on closed issues. If you think, this PR is still necessary, create a new one with the same branch. Thank you.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

bug: TRPC may create an invalid URL that crashes the server

2 participants