-
Notifications
You must be signed in to change notification settings - Fork 3.9k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Fix error when accessing some not found pages #2751
Conversation
🦋 Changeset detectedLatest commit: b20bcaa The changes in this PR will be included in the next version bump. This PR includes changesets to release 1 package
Not sure what this means? Click here to learn what changesets are. Click here if you're a maintainer who wants to add another changeset to this PR |
Fix GITBOOK-OPEN-1XDB Fix GITBOOK-OPEN-1XDC Fix GITBOOK-OPEN-1X07 Fix GITBOOK-OPEN-1XCJ Fix GITBOOK-OPEN-1X86
d07aed1
to
32305f4
Compare
GitBook Preview |
The latest updates on your projects. Learn more about Argos notifications ↗︎
|
984b230
to
3e2bbf2
Compare
Context
Some URLs are not matched by the middleware but they are matched from our main route.
In this PR we handle that gracefully, if a request is not from the middleware we will display a proper 404 instead of a 500 error.
Sentry
Fix GITBOOK-OPEN-1XDB
Fix GITBOOK-OPEN-1XDC
Fix GITBOOK-OPEN-1X07
Fix GITBOOK-OPEN-1XCJ
Fix GITBOOK-OPEN-1X86