Fix broken homepage links and YouTube embed (Error 153)#59
Merged
Conversation
Dependency Review✅ No vulnerabilities or license issues or OpenSSF Scorecard issues found.Scanned FilesNone |
The docs concepts/ section was restructured and concepts/application-graph no longer exists (404). Point the two disabled 'Learn More' buttons to concepts/applications/, which now documents the application graph. The Infrastructure Recipes link was already fixed on main in #57. Fixes #58 Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> Signed-off-by: Brooke Hamilton <45323234+brooke-hamilton@users.noreply.github.com>
brooke-hamilton
force-pushed
the
brooke-hamilton-fix-broken-links
branch
from
08df290 to
7cd52f3
Compare
brooke-hamilton
changed the title
The site sends Referrer-Policy: same-origin, which strips the referrer on cross-origin requests. Without a referrer, the YouTube player cannot verify the embedding origin and fails with 'Error 153 - Video player configuration error'. Set referrerpolicy=strict-origin-when-cross-origin on the iframe (the same value YouTube uses in its oEmbed embed code) so the player receives the origin and loads correctly. Verified with a reproduction that the embed loads after the change. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> Signed-off-by: Brooke Hamilton <45323234+brooke-hamilton@users.noreply.github.com>
brooke-hamilton
changed the title
There was a problem hiding this comment.
Pull request overview
This pull request fixes issues uncovered by automated link testing of the radapp.io homepage by updating two stale docs URLs in homepage content and adjusting the homepage YouTube iframe embed so it functions correctly under a restrictive Referrer-Policy.
Changes:
- Update two disabled “Learn More” button links from the removed
concepts/application-graphdocs page to/concepts/applications/. - Add
referrerpolicy="strict-origin-when-cross-origin"to the homepage YouTube embed iframe to prevent “Error 153” when the site usesReferrer-Policy: same-origin.
Reviewed changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.
| File | Description |
|---|---|
themes/bigspring-light/layouts/index.html |
Adds referrerpolicy to the YouTube embed iframe in the screenshot section to fix playback under restrictive referrer headers. |
content/_index.md |
Updates two homepage content links to the new docs location for application graph documentation. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
DariuszPorowski
approved these changes
Jun 19, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Fixes broken homepage items found while link-testing https://radapp.io with Playwright.
1. Broken
concepts/application-graphlinks (content/_index.md)The docs
concepts/section was restructured andconcepts/application-graphno longer exists (404). Updated both "Learn More" buttons to/concepts/applications/, which now documents the application graph (it has a dedicated "The application graph" section). Verified HTTP 200.https://docs.radapp.io/concepts/application-graphhttps://docs.radapp.io/concepts/applications/https://docs.radapp.io/concepts/application-graphhttps://docs.radapp.io/concepts/applications/(These two links are on buttons with
enable: false, so they aren't rendered on the live site, but the broken URLs were still in source.)2. YouTube embed "Error 153 – Video player configuration error" (
themes/bigspring-light/layouts/index.html)The homepage "Learn more about Radius" video failed to load with Error 153. Root cause: the site sends
Referrer-Policy: same-origin, which strips the referrer on cross-origin requests, so the YouTube player can't verify the embedding origin and refuses to play.Fix: add
referrerpolicy="strict-origin-when-cross-origin"to the embed iframe (the same value YouTube uses in its own oEmbed embed code), which overrides the document policy for that frame so the player receives the origin.Reproduced the error and verified the fix loads the video correctly before/after the change.
Verification
Crawled all 3 pages in the radapp.io sitemap (
/,/categories/,/tags/) and validated all 21 unique links — including every outbounddocs.radapp.iolink — all return HTTP 200.Closes #58