Skip to content

Update middleware.ts integrations #13888

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.

Sign up for GitHub

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

Open
masinette wants to merge 5 commits into
base: master
Choose a base branch
from

[getsentry/action-github-commit] Auto commit

703cee1
Select commit
Loading
Failed to load commit list.
Open

Update middleware.ts integrations #13888

[getsentry/action-github-commit] Auto commit
703cee1
Select commit
Loading
Failed to load commit list.
Cursor / Cursor BugBot completed Jul 18, 2025 in 2m 40s

BugBot Review

BugBot Analysis Progress (2m 42s elapsed)

✅ Gathered PR context (2s)
✅ Analyzed code changes (0s)
✅ Completed bug detection — 2 potential bugs found (2m 38s)
✅ Validation and filtering completed (0s)
✅ Posted analysis results — 2 bugs reported (2s)
✅ Analysis completed successfully (0s)

Final Result: BugBot completed review and found 2 potential issues

Request ID: serverGenReqId_b4e72257-cc55-4c2b-812a-487c189dfe9f

Details

Bug: Redirect Path Error Causes 404

The redirect for /organization/integrations/elba/ was incorrectly modified to /organization/integrations/elba-alpha/ when moved from DEVELOPER_DOCS_REDIRECTS to USER_DOCS_REDIRECTS. This change was unintended, as the PR aimed to only move redirects, not alter their paths. Consequently, existing links to the original /organization/integrations/elba/ URL will now result in 404 errors.

src/middleware.ts#L222-L225

sentry-docs/src/middleware.ts

Lines 222 to 225 in 703cee1

{
from: '/organization/integrations/elba-alpha/',
to: '/organization/integrations/compliance/elba-alpha/',
},

Fix in Cursor • Fix in Web

Bug: Removed Redirects Cause User Errors

The commit removes several functional redirects from DEVELOPER_DOCS_REDIRECTS without replacement, which will cause 404 errors for users. The affected paths include /feature-flags/options-backed-features/, /sdk/metrics/, /backend/feature-flags/options-backed-features/, and /api-server/application-domains/feature-flags/options-backed-features/.

src/middleware.ts#L3650-L3668

sentry-docs/src/middleware.ts

Lines 3650 to 3668 in 703cee1

},
{
from: '/options/',
to: '/backend/options/',
},
{
from: '/sdk/features/data-handling/',
to: '/sdk/expected-features/data-handling/',
},
{
from: '/sdk/craft-quick-start/',
to: '/sdk/processes/releases/',
},
{
from: '/sdk/check-ins/',
to: '/sdk/telemetry/check-ins/',
},
{
from: '/sdk/profiles/',

src/middleware.ts#L3860-L3868

sentry-docs/src/middleware.ts

Lines 3860 to 3868 in 703cee1

},
{
from: '/backend/options/',
to: '/backend/application-domains/options/',
},
{
from: '/api-server/application-domains/options/',
to: '/develop/application-domains/options/',
},

Fix in Cursor • Fix in Web

Was this report helpful? Give feedback by reacting with 👍 or 👎

View more details on Cursor