Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Fix require-trusted-types-for #76

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

dragonbear-os
Copy link

This is a fix for #75

I added "script" to the set of literal directive values and added a test for it.

I couldn't get the tests to run without using --runInBand which runs all the tests in a single thread which exposed that empty was getting reused across tests. So I changed the uses of empty to create a new object rather than use the one shared constant.

@changeset-bot
Copy link

changeset-bot bot commented Sep 9, 2022

🦋 Changeset detected

Latest commit: fa7c4b0

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 4 packages
Name Type
@strict-csp/builder Patch
@next-safe/middleware Patch
docs Patch
e2e Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@vercel
Copy link

vercel bot commented Sep 9, 2022

Someone is attempting to deploy a commit to a Personal Account owned by @nibtime on Vercel.

@nibtime first needs to authorize it.

@vercel
Copy link

vercel bot commented Sep 15, 2022

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Updated
docs-next-safe-middleware ✅ Ready (Inspect) Visit Preview Sep 15, 2022 at 6:02PM (UTC)

@danfsd
Copy link

danfsd commented Mar 20, 2023

@nibtime can you please check this one?

@danfsd
Copy link

danfsd commented Apr 24, 2023

It looks like @nibtime is unresponsive... @boennemann could you take a look into this PR?

I need require-trusted-types-for to work for an incoming audit and this PR made it work for me

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants