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

Jump to bottom

fix(protocol): Add missing security directives #876

Merged
merged 3 commits into from
Dec 7, 2020
Merged

fix(protocol): Add missing security directives #876

merged 3 commits into from
Dec 7, 2020

Conversation

jan-auer
Copy link
Member

@jan-auer jan-auer commented Dec 7, 2020

Several variants of CspDirective were missing, which caused us to drop CSP
reports that specified them in their effective-directive field. The list of
directives was obtained from https://developer.mozilla.org/en-US/docs/Web/HTTP/Headers/Content-Security-Policy

@jan-auer jan-auer requested review from a team and RaduW December 7, 2020 10:58
RaduW
RaduW reviewed Dec 7, 2020
View reviewed changes
CHANGELOG.md Outdated Show resolved Hide resolved
@jan-auer jan-auer merged commit bb05f0c into master Dec 7, 2020
@jan-auer jan-auer deleted the fix/missing-effective-directive branch December 7, 2020 11:40
@oioki oioki mentioned this pull request Mar 17, 2023
Closed
# for free to join this conversation on GitHub. Already have an account? # to comment
Reviewers

@RaduW RaduW RaduW left review comments

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@jan-auer @RaduW