Skip to content

Change http-csp delivery to referrer policy header#3118

Merged
jdm merged 1 commit intoweb-platform-tests:masterfrom
rebstar6:httprefpol2
Jun 7, 2016
Merged

Change http-csp delivery to referrer policy header#3118
jdm merged 1 commit intoweb-platform-tests:masterfrom
rebstar6:httprefpol2

Conversation

@rebstar6
Copy link
Copy Markdown
Contributor

@rebstar6 rebstar6 commented Jun 7, 2016

Redo of #3116

@hoppipolla-critic-bot
Copy link
Copy Markdown

Critic review: https://critic.hoppipolla.co.uk/r/6589

This is an external review system which you may optionally use for the code review of your pull request.

In order to help critic track your changes, please do not make in-place history rewrites (e.g. via git rebase -i or git commit --amend) when updating this pull request.

@wpt-pr-bot
Copy link
Copy Markdown
Collaborator

Reviewers for this pull request are: @kristijanburnik.

@rebstar6
Copy link
Copy Markdown
Contributor Author

rebstar6 commented Jun 7, 2016

@jdm - to your comment on the last one - meta-csp is for subresources. It should be changed to use referrer, but I think the structure can be kept, so I don't want to take it out until I look at that piece.

In either case, keeping it in, should be edited in a separate PR.

@jdm
Copy link
Copy Markdown
Contributor

jdm commented Jun 7, 2016

Makes sense!

@jdm jdm merged commit 2c0f2a9 into web-platform-tests:master Jun 7, 2016
@jdm
Copy link
Copy Markdown
Contributor

jdm commented Jun 7, 2016

Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants