Issue metadata
Sign in to add a comment
|
Network.setExtraHTTPHeaders no longer adds headers to initial navigation request |
||||||||||||||||||||||||
Issue descriptionLooks like 63 introduced a bug where the Chrome remote debug protocol Network.setExtraHTTPHeaders is no longer adding the headers to the initial navigation request. My guess is that it is because of plznavigate and the navigation request being initiated from the browser and not the renderer. For example, this webpagetest test: https://www.webpagetest.org/result/171207_1H_647bfdafac28e369757b0b6837595fe1/ adds a custom header "x-Bob: CustomHeaderHere" by sending the message: {"params":{"headers":{"x-Bob":"CustomHeaderHere"}},"id":9,"method":"Network.setExtraHTTPHeaders"} Looking through the individual requests the header is applied to all of the requests except for the first navigation request. Remote debug protocol spec: https://chromedevtools.github.io/devtools-protocol/tot/Network/#method-setExtraHTTPHeaders
,
Dec 7 2017
,
Dec 7 2017
,
Dec 7 2017
Note --disable-browser-side-navigation is going away next week as we start removing the old navigation code path (which is unsupported with Chrome at this point).
,
Dec 8 2017
,
Dec 8 2017
This is already fixed in m64 (see issue 767683 for details) |
|||||||||||||||||||||||||
►
Sign in to add a comment |
|||||||||||||||||||||||||
Comment 1 by pmeenan@chromium.org
, Dec 7 2017