Issue metadata
Sign in to add a comment
|
chrome.webRequest.onBeforeSendHeaders can't change Referer in Chrome 71
Reported by
human.p...@gmail.com,
Dec 9
|
||||||||||||||||||||||
Issue descriptionUserAgent: Mozilla/5.0 (Windows NT 6.1; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/71.0.3578.80 Safari/537.36 Steps to reproduce the problem: 1. Load the attached unpacked extension. 2. Try to open any website and observe the headers. For example, https://nghttp2.org/httpbin/headers What is the expected behavior? A referer header should be added with value "Test". What went wrong? It isn't. According to https://developer.chrome.com/extensions/webRequest, you need to add "extraHeaders" option to modify Referer starting from 72, but that shouldn't be the case for 71. Actually, 71 doesn't even support "extraHeaders" option yet (will throw error). Did this work before? Yes Does this work in other browsers? Yes Chrome version: 71.0.3578.80 Channel: stable OS Version: 6.1 (Windows 7, Windows Server 2008 R2) Flash Version:
,
Dec 9
,
Dec 10
Tried testing the issue on chrome reported version# 71.0.3578.80 using windows-10 with steps mentioned below: 1) Launched chrome reported version and loaded the extension file in chrome://extensions 2) Navigated to page: https://nghttp2.org/httpbin/headers, seen same bahaviour on stable and beta, find attached screencast for reference @Reporter: Please find the attached screencast for your reference and let us know if we missed anything in reproducing the issue, provide your feedback on it which help in further triaging it in better way. Thanks!
,
Dec 10
Thanks for the testing. I can't spot anything wrong in your sreencat, it has the exact steps as I had. The only difference is you're testing on Win 10, and I did it on Win 7. It might be related to one of many A/B test switches, too.
,
Dec 10
Thank you for providing more feedback. Adding the requester to the cc list. For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
,
Dec 10
,
Dec 10
In order to see which experiment is enabled in which profile, you can go to - chrome://version/?show-variations-cmd You can compare the output when you have this issue and when you do not have this issue and report back.
,
Dec 10
Thanks! Not sure how to compare, but I pasted them here: https://gist.github.com/fireattack/346caeeede5a32c4741c13078540a6e2
,
Dec 10
I replaced every , with ,\n and every / with /\n and compared using https://www.diffnow.com/ - I found that NetworkService is enabled in the bad one, which makes sense (I would be surprised if it were enabled by default for some users on stable, though, but maybe the development is mostly done). Why does it make sense? Because this NetworkService experiment is a radical change to the network stack (moving it out of the browser process and into its own process, if I understand correctly) and webRequest is, well, very related to the network stack. Also, this was announced, as it turns out - https://groups.google.com/a/chromium.org/d/msgid/chromium-extensions/bbac59f9-ea9e-4e1d-83b8-8341fe0dec50%40chromium.org?utm_medium=email&utm_source=footer
,
Dec 10
>Also, this was announced, as it turns out I mentioned this in my initial report already: "According to https://developer.chrome.com/extensions/webRequest, you need to add "extraHeaders" option to modify Referer starting from 72, but that shouldn't be the case for 71." It was announced to be changed in 72, not in 71, isn't it? And I can't use the "new" way either because 'extraHeaders' is not supported in this version (which I also mentioned). "Actually, 71 doesn't even support "extraHeaders" option yet (will throw error)."
,
Dec 10
All in all: I understand the situation; but is there anyway to make it work in my Beta Chrome 71? Like manually turn off this experiment? Thanks.
,
Dec 10
Bummer :( Try going to chrome://flags/#network-service and disabling it.
,
Dec 10
Thank you, will give it a try later. |
|||||||||||||||||||||||
►
Sign in to add a comment |
|||||||||||||||||||||||
Comment 1 by human.p...@gmail.com
, Dec 9