We would like the URls that have a token to have the x-robots-tag set to "noindex".
| Comment | File | Size | Author |
|---|---|---|---|
| #13 | interdiff-3130346-12-13.txt | 1 KB | chr.fritsch |
| #13 | 3130346-13.patch | 10.1 KB | chr.fritsch |
| #12 | interdiff-3130346-11-12.txt | 3.06 KB | chr.fritsch |
| #12 | 3130346-12.patch | 9.04 KB | chr.fritsch |
| #11 | 3130346-11.patch | 6.43 KB | chr.fritsch |
Comments
Comment #2
killes@www.drop.org commentedsomething like this
Comment #3
sharma.amitt16 commentedComment #4
sharma.amitt16 commentedkilles@www.drop.org, I agree with the request. But in your code, you are adding header in the foreach() loop and fixed the headers in the array. But others may want to add more headers on such pages. So we can't hardcode this in the code itself.
So I have added a field on settings form for that so one can configure these from the backend. Not it is allowed to add multiple headers on settings form (admin/config/content/access_unpublished) and headers will be reflected on-page.
Comment #6
sharma.amitt16 commentedFix for failed test cases.
Comment #7
Rkumar commentedComment #8
sharma.amitt16 commented@killes@www.drop.org @martin_klima Would you guys please review this?
Comment #9
chr.fritschThanks for the work. I cleaned everything up a bit. Update hook is still missing
Comment #10
chr.fritschWith update hook
Comment #11
chr.fritschRerolled
Comment #12
chr.fritschAnd now with a test
Comment #13
chr.fritschUh, we have an error in D8.9. Lets quickly fix it
Comment #14
daniel.bosenlgtm
Comment #16
chr.fritschDone