Thanks to visit codestin.com
Credit goes to github.com

Skip to content

fix: youtube embedded videos not showing in /education (#4730) #4770

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

Jump to bottom
Merged
DonnieBLT merged 1 commit into from
Nov 12, 2025
Merged

fix: youtube embedded videos not showing in /education (#4730) #4770

DonnieBLT merged 1 commit into from
Nov 12, 2025
+5 −0

Conversation

@nileshpahari
Copy link
Contributor

@nileshpahari nileshpahari commented Nov 10, 2025 •
edited by coderabbitai bot
Loading

Fixes: #4730
Embedded YouTube videos were showing 'Error 153'
Added the attribute referrerpolicy="strict-origin-when-cross-origin" to all <iframe> elements to resolve the issue (as also suggested by other contributors).
image
image

Summary by CodeRabbit

  • Bug Fixes
    • Updated embedded YouTube videos in the Featured videos section to use a stricter referrer policy, improving privacy and security when playing videos. This change does not alter playback behavior or layout—only how referrer information is handled when navigating to embedded content.

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Nov 10, 2025 •
edited
Loading

Walkthrough

Added referrerpolicy="strict-origin-when-cross-origin" attributes to multiple YouTube iframe embeds in the education template's featured videos/lectures section; no other structural or behavioral changes.

Changes

Cohort / File(s) Summary
Security Policy Enhancement
website/templates/education/education.html		 Added referrerpolicy="strict-origin-when-cross-origin" attributes to YouTube iframe embeds in the featured videos/lectures section to restrict referrer header information.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~8 minutes

  • Verify all YouTube iframe elements in website/templates/education/education.html include the new referrerpolicy attribute.
  • Confirm consistent application across all embed instances and that embeds still load as expected.

Pre-merge checks and finishing touches

✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title clearly and specifically identifies the fix for the YouTube video embedding issue in the education section, directly matching the main change in the PR.
Linked Issues check ✅ Passed The PR addresses the linked issue #4730 by adding referrerpolicy attribute to iframe elements to resolve YouTube Error 153, implementing the identified fix.
Out of Scope Changes check ✅ Passed All changes are limited to adding referrerpolicy attributes to YouTube iframes in education.html, staying within scope of fixing the embedded video issue.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
📜 Recent review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

Knowledge base: Disabled due to Reviews -> Disable Knowledge Base setting

📥 Commits

Reviewing files that changed from the base of the PR and between 151ba25 and f8f64ed.

📒 Files selected for processing (1)
  • website/templates/education/education.html (5 hunks)
🚧 Files skipped from review as they are similar to previous changes (1)
  • website/templates/education/education.html
⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (2)
  • GitHub Check: Run Tests
  • GitHub Check: docker-test

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@github-project-automation github-project-automation bot moved this from Backlog to Ready in 📌 OWASP BLT Project Board Nov 10, 2025
@nileshpahari nileshpahari force-pushed the fix/videoplayer-error-4730 branch from 151ba25 to f8f64ed Compare November 10, 2025 13:17
@DonnieBLT DonnieBLT merged commit e363356 into OWASP-BLT:main Nov 12, 2025
15 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@DonnieBLT DonnieBLT DonnieBLT approved these changes

@coderabbitai coderabbitai[bot] coderabbitai[bot] approved these changes

Assignees

No one assigned

Labels

None yet

Projects

📌 OWASP BLT Project Board
Status: Done

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Youtube embeded videos aren't working in education section.

2 participants

@nileshpahari @DonnieBLT