I've added support for the newish Referrer-Policy header using the attached patch. https://securityheaders.io seems to accept the results. If it's agreed that this is a useful addition to the project, I'll see what I can do with the 8.x-dev version in the next day or two.
| Comment | File | Size | Author |
|---|---|---|---|
| #12 | 2925376-12-referrer_policy.patch | 5.55 KB | adammalone |
| #8 | seckit-8-referrer-policy-2925376-8.patch | 7.37 KB | mcdruid |
| #5 | seckit-8-referrer-policy-2925376.patch | 7.74 KB | lamp5 |
| #2 | seckit-8-referrer-policy.patch | 7.51 KB | wrd |
| seckit-referrer-policy.patch | 6.44 KB | wrd |
Comments
Comment #2
wrd commentedTurned out I needed it myself for D8, so here is a patch for the 8.x-dev branch.
Comment #3
wrd commentedComment #4
wrd commentedUpdated to 8.x-1.x-dev, as that's probably more relevant.
Comment #5
lamp5Hi. Thanks, @wrd for the patch. But I have a problem to apply it using composer and last dev. Reroll attached.
Comment #7
hsponner commentedI have the same problem as #5, can't apply the patch for latest dev version.
Comment #8
mcdruid commentedThanks. I've applied the codesniffer fixes and tidied a couple of other things up.
Re-testing before committing this.
Comment #10
mcdruid commentedThanks! Committed to 8.x-1.x
Leaving this open for backport to D7.
Comment #11
hsponner commentedThanks @mcdruid, works like a charm.
Comment #12
adammaloneAdding D7 patch here for review.
Comment #14
mcdruid commentedThanks Adam! I tweaked some whitespace and a couple of arrays to use the retro syntax for D7, but didn't change anything significant in your patch.
Thank you all who contributed to this!