fix(Docs) redirect and deprecate old metrics pages by sergical · Pull Request #14602 · getsentry/sentry-docs

sergical · 2025-08-08T19:03:50Z

DESCRIBE YOUR PR

Deprecate and redirect old metrics pages
Remove bandaid fix for Redirects failing on prod #11870 (reverts Fix bad link and update broken redirects #11878)

IS YOUR CHANGE URGENT?

Help us prioritize incoming PRs by letting us know when the change needs to go live.

Urgent deadline (GA date, etc.):
Other deadline:
None: Not urgent, can wait up to 1 week+

SLA

Teamwork makes the dream work, so please add a reviewer to your PRs.
Please give the docs team up to 1 week to review your PR unless you've added an urgent due date to it.
Thanks in advance for your help!

PRE-MERGE CHECKLIST

Make sure you've checked the following before merging your changes:

Checked Vercel preview for correctness, including links
PR was reviewed and approved by any necessary SMEs (subject matter experts)
PR was reviewed and approved by a member of the Sentry docs team

LEGAL BOILERPLATE

Look, I get it. The entity doing business as "Sentry" was incorporated in the State of Delaware in 2015 as Functional Software, Inc. and is gonna need some rights from me in order to utilize my contributions in this here PR. So here's the deal: I retain all rights, title and interest in and to my contributions, and by keeping this boilerplate intact I confirm that Sentry can use, modify, copy, and redistribute my contributions, under Sentry's choice of terms.

EXTRA RESOURCES

Sentry Docs contributor guide

linear · 2025-08-08T19:03:53Z

DEVEX-102 deprecated generic metrics pages

vercel · 2025-08-08T19:03:57Z

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

Name	Status	Preview	Comments	Updated (UTC)
develop-docs	✅ Ready (Inspect)	Visit Preview	💬 Add feedback	Aug 8, 2025 10:49pm
sentry-docs	✅ Ready (Inspect)	Visit Preview	💬 Add feedback	Aug 8, 2025 10:49pm

codecov · 2025-08-08T19:10:53Z

Bundle Report

Changes will decrease total bundle size by 566 bytes (-0.0%) ⬇️. This is within the configured threshold ✅

Detailed changes

Bundle name	Size	Change
sentry-docs-edge-server-array-push	351.69kB	142 bytes (0.04%) ⬆️
sentry-docs-client-array-push	9.85MB	-6 bytes (-0.0%) ⬇️
sentry-docs-server-cjs	12.37MB	-702 bytes (-0.01%) ⬇️

Affected Assets, Files, and Routes:

view changes for bundle: sentry-docs-edge-server-array-push

Assets Changed:

Asset Name	Size Change	Total Size	Change (%)
`src/middleware.js`	142 bytes	206.97kB	0.07%

view changes for bundle: sentry-docs-client-array-push

Assets Changed:

Asset Name	Size Change	Total Size	Change (%)
`static/chunks/pages/_app-*.js`	-3 bytes	882.71kB	-0.0%
`static/chunks/8321-*.js`	-3 bytes	425.87kB	-0.0%
`server/middleware-*.js`	5.55kB	6.55kB	555.3% ⚠️
`server/middleware-*.js`	-5.55kB	1.0kB	-84.74%
`static/k4k4zm8BolyAGsPtCwT7o/_buildManifest.js`(New)	684 bytes	684 bytes	100.0% 🚀
`static/k4k4zm8BolyAGsPtCwT7o/_ssgManifest.js`(New)	77 bytes	77 bytes	100.0% 🚀
~~`static/Y_Ge8RttZxzBllCkofes7/_buildManifest.js`~~(Deleted)	-684 bytes	0 bytes	-100.0% 🗑️
~~`static/Y_Ge8RttZxzBllCkofes7/_ssgManifest.js`~~(Deleted)	-77 bytes	0 bytes	-100.0% 🗑️

view changes for bundle: sentry-docs-server-cjs

Assets Changed:

Asset Name	Size Change	Total Size	Change (%)
`1729.js`	-3 bytes	1.77MB	-0.0%
`../instrumentation.js`	-3 bytes	1.1MB	-0.0%
`9523.js`	-3 bytes	1.08MB	-0.0%
`../app/[[...path]]/page.js.nft.json`	-231 bytes	744.23kB	-0.03%
`../app/platform-redirect/page.js.nft.json`	-231 bytes	744.14kB	-0.03%
`../app/sitemap.xml/route.js.nft.json`	-231 bytes	741.61kB	-0.03%

seer-by-sentry · 2025-08-08T22:31:24Z

+ to: '//concepts/key-terms/tracing/span-metrics/',
 },


Potential bug: The Android metrics redirect URL in `src/middleware.ts` has an extra leading slash, causing it to become a protocol-relative URL and redirect to a non-existent external domain.
Description: The redirect configuration for /platforms/android/metrics/ incorrectly specifies the destination as //concepts/key-terms/tracing/span-metrics/. The double slash // causes the URL constructor to create a protocol-relative URL. This results in an attempt to redirect users to an external, non-existent domain like https://concepts/... instead of the intended internal documentation page. This breaks the redirect functionality for users trying to access the deprecated Android metrics documentation.
Suggested fix: In src/middleware.ts, change the to value for the Android metrics redirect from '//concepts/key-terms/tracing/span-metrics/' to '/concepts/key-terms/tracing/span-metrics/'. This will make it a standard relative path, consistent with other redirects in the file.
_{severity: 0.6, confidence: 0.95}
_{Did we get this right? 👍 / 👎 to inform future reviews.}

chargome

🥇

* 'master' of https://github.com/getsentry/sentry-docs: (22 commits) Fix incorrectly placed comma (#14592) fix(devservices): Typo in devservices docs (#14636) ref(ourlogs): Add log size limit (#14635) Document undocumented php prefix option (#14634) getsentry/relay@7958273 Unreal Engine: Update plugin docs (#14624) feat: Update min js sdk version for logging and update console logging snippet (#14614) feat: [Unreal] Add transaction and span set_data documentation (#14599) feat(js): Add `ignoreSpans` documentation to Options page (#14572) fix(js/browser): Amend user IP + `sendDefaultPii` change to v10 migration guide (#14609) fix: Fix broken AI banner due to invalid syntax (#14627) feat(partners)add-enablement-docs (#14095) Fix image path issue & add test coverage for new lightbox (#14616) feat(seer): update GHE setup to work with seer (#14617) fix(replay): Document searchable properties `screen` and alias `screens` (#14603) Update image handling (#14564) Update using-hooks.mdx (#14605) Godot: Document `disabled-in-editor-play` option (#14438) fix(Docs) redirect and deprecate old metrics pages (#14602) fix(relay): Update projects.mdx (#14606) ...

fix(Docs) redirect and deprecate old metrics pages
cd32451

[getsentry/action-github-commit] Auto commit
8f2eb96

vercelBotdeployed to Preview – develop-docs August 8, 2025 19:15 View deployment

vercelBotdeployed to Preview – sentry-docs August 8, 2025 19:46 View deployment

remove other pages
be66861

codeowner-assignmentBot requested review from a teamAugust 8, 2025 20:42

[getsentry/action-github-commit] Auto commit
dfb6873

codeowner-assignmentBot requested review from a teamAugust 8, 2025 20:43

vercelBotdeployed to Preview – develop-docs August 8, 2025 20:53 View deployment

vercelBotdeployed to Preview – sentry-docs August 8, 2025 21:14 View deployment

sergical added 2 commits August 8, 2025 17:42

updates
c02695f

update redirects
7d0f8e0

vercelBotdeployed to Preview – develop-docs August 8, 2025 21:52 View deployment

vercelBotdeployed to Preview – sentry-docs August 8, 2025 22:11 View deployment

sergical marked this pull request as ready for review August 8, 2025 22:27

This comment was marked as outdated.
Sign in to view

seer-by-sentryBot reviewed Aug 8, 2025
View reviewed changes

thank you 🤖
3596df3

cursorBot reviewed Aug 8, 2025
View reviewed changes

Comment threadsrc/middleware.ts
Comment threadsrc/middleware.ts

vercelBotdeployed to Preview – develop-docs August 8, 2025 22:40 View deployment

vercelBotdeployed to Preview – sentry-docs August 8, 2025 22:49 View deployment

chargome approved these changes Aug 10, 2025
View reviewed changes

sergical merged commit 731278e into masterAug 11, 2025
15 checks passed

sergical deleted the sergiydybskiy/devex-102-deprecated-generic-metrics-pages branch August 11, 2025 13:51

cleptric mentioned this pull request Aug 13, 2025
chore(metrics): Clean up metrics CLI docs #14654
Merged

github-actionsBot locked and limited conversation to collaborators Aug 27, 2025

Navigation Menu

Uh oh!

Uh oh!

fix(Docs) redirect and deprecate old metrics pages - #14602

fix(Docs) redirect and deprecate old metrics pages#14602
sergical merged 7 commits into
masterfrom
sergiydybskiy/devex-102-deprecated-generic-metrics-pages

sergical commented Aug 8, 2025•
edited
Loading

Uh oh!

linearBot commented Aug 8, 2025

Uh oh!

vercelBot commented Aug 8, 2025•
edited
Loading

Uh oh!

codecovBot commented Aug 8, 2025•
edited
Loading

Assets Changed:

Assets Changed:

Assets Changed:

Uh oh!

This comment was marked as outdated.

Uh oh!

seer-by-sentryBotAug 8, 2025

Uh oh!

Uh oh!

Uh oh!

chargome left a comment

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

Uh oh!

Uh oh!

Conversation

sergical commented Aug 8, 2025• edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

DESCRIBE YOUR PR

IS YOUR CHANGE URGENT?

SLA

PRE-MERGE CHECKLIST

LEGAL BOILERPLATE

EXTRA RESOURCES

Uh oh!

linearBot commented Aug 8, 2025

Uh oh!

vercelBot commented Aug 8, 2025• edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

codecovBot commented Aug 8, 2025• edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Bundle Report

Affected Assets, Files, and Routes:

Assets Changed:

Assets Changed:

Assets Changed:

Uh oh!

This comment was marked as outdated.

Uh oh!

seer-by-sentryBotAug 8, 2025

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

chargome left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

sergical commented Aug 8, 2025•
edited
Loading

vercelBot commented Aug 8, 2025•
edited
Loading

codecovBot commented Aug 8, 2025•
edited
Loading