Skip to content

Security hardening + CSP compatibility fixes: Twitter embeds, Spotify popup, inline style - #270

Merged
lpmi-13 merged 4 commits into
mainfrom
copilot/clone-pull-252
Aug 2, 2026
Merged

Security hardening + CSP compatibility fixes: Twitter embeds, Spotify popup, inline style#270
lpmi-13 merged 4 commits into
mainfrom
copilot/clone-pull-252

Conversation

Copilot AI commented Aug 2, 2026

Copy link
Copy Markdown
Contributor

Builds on the security changes from #252 (strict CSP, Bunny Fonts, extracted inline scripts, iframe sandbox) with three fixes for functional regressions introduced by the strict CSP.

Fixes

  • Twitter embeds (_hacks/11.md)script-src 'self' blocked platform.twitter.com/widgets.js, leaving blockquotes unstyled. Added platform.twitter.com to script-src/frame-src, pbs.twimg.com/abs.twimg.com to img-src, and syndication.twitter.com to connect-src.

  • Hack time bold (_layouts/hack.html)style="font-weight: bold;" silently dropped by style-src without 'unsafe-inline'. Replaced with a .hack-time class in _sass/_style.scss.

  • Spotify "Open in Spotify" (pages/podcast.md) — missing allow-popups in iframe sandbox prevented new-tab navigation. Added it alongside the existing allow-scripts allow-same-origin allow-presentation.

Copilot AI and others added 4 commits February 25, 2026 20:02
…unny Fonts, iframe sandbox

Co-authored-by: benfoxall <51385+benfoxall@users.noreply.github.com>
…ost:26831/remotehack/remotehack.github.io into copilot/clone-pull-252
Copilot AI changed the title Security hardening: CSP, Referrer Policy, Bunny Fonts, iframe sandbox (with site-functionality fixes) Security hardening + CSP compatibility fixes: Twitter embeds, Spotify popup, inline style Aug 2, 2026
Copilot AI requested a review from lpmi-13 August 2, 2026 17:36
@lpmi-13
lpmi-13 marked this pull request as ready for review August 2, 2026 17:36
@lpmi-13
lpmi-13 merged commit 2a33ed1 into main Aug 2, 2026
@lpmi-13
lpmi-13 deleted the copilot/clone-pull-252 branch August 2, 2026 17:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants