-
Notifications
You must be signed in to change notification settings - Fork 10.2k
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
[Chrome extension] Add Referer request header if needed #5823
Conversation
Do you have steps on how to verify this feature? Most importantly a website that requires a referrer. Perhaps @Hengjie could also test this? |
Steps to verify:
|
Ping, could anyone review this? |
I'm sorry to put this on @yurydelendik's review stack, but I do not know enough about this part of the codebase to properly review this (other than testing it of course). There does appear to be a merge conflict. Perhaps that should be resolved first? |
56d9623
to
457a076
Compare
Rebased PR. |
@timvandermeij @Rob--W I can confirm it's fixed, thanks! I tested it by seting up a nginx server locally that prevented "hotlinking" by checking the referral. |
Ping. |
@Rob--W looks good, can you merge this in? |
@yurydelendik Sure, I'll do this over the weekend. |
This patch adds the Referer request header to PDF requests if the original PDF request included the Referer header.
457a076
to
adb2f8a
Compare
Rebased and resolved trivial merge conflict (caused by an intermediate refactor of web/pdf_history.js that affected the indentation of the whole file). Using the steps in #5823 (comment), I also verified that the patch still works in Chromium 42.0.2311.90 and 45.0.2416.0. |
[Chrome extension] Add Referer request header if needed
Thank you for the patch. |
This patch adds the Referer request header to PDF requests if the original PDF request included the Referer header.
Fixes #5676.