[Doc] Fix Youtube integrations #11053
Merged
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.
Problem
Youtube integrations in the doc pages show an error and invite users to view the video on youtube instead.
Solution
This seems to be because the 'Referer' header is not passed. Adding the
referrerpolicy="strict-origin-when-cross-origin"attribute directly to the YouTube iframes seems to fix the issue.How To Test
docs/_layouts/default.htmlfile and add the following tag to the header to reproduce the issue:<meta name="referrer" content="no-referrer" />. Then, make sure the videos are properly loaded thanks to the fix.referrerpolicyattribute like this PR does. Then duplicate the iframe node, and confirm this time the video is properly loaded.Additional Checks
masterfor a bugfix or a documentation fix, ornextfor a feature