Add blob urls to Content Security Policy headers #2203
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.
This PR is to allow HTML/JS inscriptions to use Blob urls by adding it to the Content Security Policy headers.
Why is this needed?
I am creating a javascript inscription where I draw some stuff onto a canvas, then convert that canvas into a HTML img. I have successfully done this with the following JS code:
This works because
data:
is included in the Content Security Policy, so data urls are allowed. However, many sources state that blob urls should be used over data urls because they are more performant. So, I would like to do something like this:However, this gets blocked because blob urls are not included in the Content Security Policy.
Two errors get reported from this:
This is a very simple change to add
blob:
to the Content Security Policy, which resolves these errors.I have run all tests and they pass. I have also run the server locally and verified that the above code inscription runs successfully after the changes.