Skip to content
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.

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

enhance(play): allow {blob:,data:} as frame-src for live samples #9801

Closed
wants to merge 1 commit into from

Conversation

caugner
Copy link
Contributor

@caugner caugner commented Oct 11, 2023

Summary

Fixes #9360.

Problem

Live samples are now executed in the playground, but the playground has a more restrictive Content Security Policy, preventing some live examples to no longer work.

Solution

Allow blob: and data: content sources as frame-src for live samples.

Note: Expanding the live sample in the full-page playground will not work.


How did you test this change?

TBD

@caugner caugner requested a review from a team October 11, 2023 14:09
@github-actions github-actions bot added the idle label Jan 24, 2024
@github-actions github-actions bot added the merge conflicts 🚧 Please rebase onto or merge the latest main. label Mar 28, 2024
Copy link
Contributor

This pull request has merge conflicts that must be resolved before it can be merged.

@github-actions github-actions bot removed the idle label Mar 29, 2024
@github-actions github-actions bot added the idle label Apr 28, 2024
@caugner
Copy link
Contributor Author

caugner commented May 20, 2024

Closing, as we decided against this approach for security reasons.

@caugner caugner closed this May 20, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cloud-function idle merge conflicts 🚧 Please rebase onto or merge the latest main.
Projects
None yet
1 participant