-
Notifications
You must be signed in to change notification settings - Fork 2k
feat: Add script-src-elem option to CSP options. #9722
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
base: 4.7
Are you sure you want to change the base?
Conversation
* fix: Add missing translation * docs: Update changelog * fix: Apply suggestions * fix: Restore and mark as deprecated `forNoHiddenValue()` * fix: Minor typo
|
Hi there, mark-unwin! 👋 Thank you for sending this PR! We expect the following in all Pull Requests (PRs).
Important We expect all code changes or bug-fixes to be accompanied by one or more tests added to our test suite to prove the code works. If pull requests do not comply with the above, they will likely be closed. Since we are a team of volunteers, we don't have any more time to work See https://github.com/codeigniter4/CodeIgniter4/blob/develop/contributing/pull_request.md Sincerely, the mergeable bot 🤖 |
Signed-off-by: Mark Unwin <mark.unwin@gmail.com>
Signed-off-by: Mark Unwin <mark.unwin@gmail.com>
|
Please do not modify the initial PR template with checkboxes. |
|
Please make sure each pull request only contains commits for its specific change. Commits from other, already merged PRs should not be included. |
|
👋 Hi, @mark-unwin! |
|
Please fix the merge conflicts. |
|
Hey @mark-unwin, are you still interested in this feature? If yes, could you update this PR in the next two weeks? |
Added a new CSP3 directive for Content Security Policy for script-src-elem.