1565242 - nsHttpConnectionMgr::OnMsgReclaimConnection called with wrong connection entry

Assignee

Description

•

5 years ago

We are not able to remove closed connections to h2 proxies because of mixture of connection entries for anon and non-anon connections. I have a log where we create multiple connections to a proxy (note please that the limit is unreasonably 6, not 32 as it appears from the log...) where 1.1-forced and websocket CLOSED conns are not removed from the active list.

The reason is that we call OnMsgReclaimConnection with a different connection entry - we use only one entry for anonymus and onymous requests (we share them), while two entries (a- and not-a-) can coexist when using h2 (or h2 proxy).

Hence, we need to lookup the right entry at nsHttpConnectionMgr.cpp.

W/o it we just queue new requests for websockets indefinitely, when the proxy doesn't allow extended connect protocol, because here we return false: Http2Session.cpp - mozsearch

This is quite a racy thing to happen, apparently, as I discovered this only today.

Fallout from bug 1363284

Honza Bambas (:mayhemer)

Assignee

Updated

•

5 years ago

Priority: -- → P1

BugBot [:suhaib / :marco/ :calixte]

Updated

•

5 years ago

Keywords: regression

Honza Bambas (:mayhemer)

Assignee

Comment 1

•

5 years ago

STR (100%):

run and setup the client for https://github.com/mayhemer/node_http2_proxy at default config
go to https://www.websocket.org/echo.html (a simple web socket test)
let fully load
click [connect]
click [send]
expected: log shows connected and sent/received lines
wait ~30 seconds
ctrl-F5 reload the tab with the websocket test
let fully load
click [connect]

AR:

no CONNECTED log, just nothing happens, after around 60s an error shows

ER:

connected, and then you can send and receive messages as before the reload

Honza Bambas (:mayhemer)

Assignee

Comment 2

•

5 years ago

Attached file Bug 1565242 - Remove HTTP/2 connection (e2e or proxy) from the opposite (an)onymous connection entry in case it cannot be found in the entry the connection is associated with because of HTTP/2 (an)onymous connection pool sharing, r=kershaw! (deleted) — Details

Honza Bambas (:mayhemer)

Assignee

Updated

•

5 years ago

Assignee: nobody → honzab.moz

Status: NEW → ASSIGNED

Honza Bambas (:mayhemer)

Assignee

Comment 3

•

5 years ago

The patch fixes the STR from comment 1 for me.

Pulsebot

Comment 4

•

5 years ago

Pushed by honzab.moz@firemni.cz: https://hg.mozilla.org/integration/autoland/rev/27196ff54fd7 Remove HTTP/2 connection (e2e or proxy) from the opposite (an)onymous connection entry in case it cannot be found in the entry the connection is associated with because of HTTP/2 (an)onymous connection pool sharing, r=kershaw

Bogdan Tara[:bogdan_tara | bogdant]

Comment 5

•

5 years ago

Backed out changeset 27196ff54fd7 (Bug 1565242) for failing an assertion on nsHttpConnectionMgr.cpp

Push with failures: https://treeherder.mozilla.org/#/jobs?repo=autoland&searchStr=linux%2Cx64%2Cdebug%2Cfirefox%2Cfunctional%2Ctests%2C%28remote%29%2Ctest-linux64%2Fdebug-firefox-ui-functional-remote-e10s%2Cfxfn-r%28en-us%29&tochange=bbfdc264f3e0ec3b5a6a19760fdb7bb7a82b8c94&fromchange=71000a40a923067a32293ef3a01d515bbd864eed&selectedJob=256799409

Backout link: https://hg.mozilla.org/integration/autoland/rev/bbfdc264f3e0ec3b5a6a19760fdb7bb7a82b8c94

Failure log: https://treeherder.mozilla.org/logviewer.html#/jobs?job_id=256799409&repo=autoland&lineNumber=6700

[task 2019-07-16T19:54:14.843Z] 19:54:14 [task 2019-07-16T19:54:14.844Z] 19:54:14 [task 2019-07-16T19:54:14.848Z] 19:54:14 [task 2019-07-16T19:54:14.849Z] 19:54:14 [task 2019-07-16T19:54:14.853Z] 19:54:14 [task 2019-07-16T19:54:14.861Z] 19:54:14 [task 2019-07-16T19:54:15.160Z] 19:54:15 [task 2019-07-16T19:54:15.218Z] 19:54:15 [task 2019-07-16T19:54:15.219Z] 19:54:15 [task 2019-07-16T19:54:15.221Z] 19:54:15 [task 2019-07-16T19:54:15.222Z] 19:54:15 [task 2019-07-16T19:54:15.223Z] 19:54:15 [task 2019-07-16T19:54:15.224Z] 19:54:15 [task 2019-07-16T19:54:15.226Z] 19:54:15 [task 2019-07-16T19:54:15.230Z] 19:54:15 [task 2019-07-16T19:54:15.231Z] 19:54:15 [task 2019-07-16T19:54:15.231Z] 19:54:15 [task 2019-07-16T19:54:15.235Z] 19:54:15 [task 2019-07-16T19:54:15.236Z] 19:54:15 [task 2019-07-16T19:54:15.236Z] 19:54:15 [task 2019-07-16T19:54:15.239Z] 19:54:15 ...
[task 2019-07-16T19:54:15.346Z] 19:54:15 [task 2019-07-16T19:54:15.362Z] 19:54:15 [task 2019-07-16T19:54:15.379Z] 19:54:15 [task 2019-07-16T19:54:15.379Z] 19:54:15 [task 2019-07-16T19:54:15.380Z] 19:54:15 [task 2019-07-16T19:54:15.401Z] 19:54:15 [task 2019-07-16T19:54:15.401Z] 19:54:15 [task 2019-07-16T19:54:15.418Z] 19:54:15 [task 2019-07-16T19:54:15.418Z] 19:54:15 [task 2019-07-16T19:54:15.418Z] 19:54:15 [task 2019-07-16T19:54:15.418Z] 19:54:15 [task 2019-07-16T19:54:15.438Z] 19:54:15 [task 2019-07-16T19:54:15.442Z] 19:54:15 [task 2019-07-16T19:54:15.458Z] 19:54:15 [task 2019-07-16T19:54:15.479Z] 19:54:15 [task 2019-07-16T19:54:15.486Z] 19:54:15 [task 2019-07-16T19:54:15.503Z] 19:54:15 [task 2019-07-16T19:54:15.503Z] 19:54:15 [task 2019-07-16T19:54:15.503Z] 19:54:15 [task 2019-07-16T19:54:15.503Z] 19:54:15 [task 2019-07-16T19:54:15.612Z] 19:54:15 [task 2019-07-16T19:54:15.628Z] 19:54:15 [task 2019-07-16T19:54:15.709Z] 19:54:15 [task 2019-07-16T19:54:15.711Z] 19:54:15 [task 2019-07-16T19:54:15.712Z] 19:54:15 [task 2019-07-16T19:54:15.717Z] 19:54:15 [task 2019-07-16T19:54:15.718Z] 19:54:15 [task 2019-07-16T19:54:15.720Z] 19:54:15 [task 2019-07-16T19:54:15.720Z] 19:54:15 [task 2019-07-16T19:54:15.727Z] 19:54:15 [task 2019-07-16T19:54:15.748Z] 19:54:15 [task 2019-07-16T19:54:15.765Z] 19:54:15 [task 2019-07-16T19:54:15.765Z] 19:54:15 [task 2019-07-16T19:54:15.765Z] 19:54:15 [task 2019-07-16T19:54:15.765Z] 19:54:15 [task 2019-07-16T19:54:15.765Z] 19:54:15 [task 2019-07-16T19:54:15.765Z] 19:54:15 [task 2019-07-16T19:54:15.766Z] 19:54:15 [task 2019-07-16T19:54:15.768Z] 19:54:15 [task 2019-07-16T19:54:15.769Z] 19:54:15 [task 2019-07-16T19:54:15.769Z] 19:54:15 [task 2019-07-16T19:54:15.771Z] 19:54:15 [task 2019-07-16T19:54:15.771Z] 19:54:15 [task 2019-07-16T19:54:15.772Z] 19:54:15 [task 2019-07-16T19:54:15.773Z] 19:54:15 [task 2019-07-16T19:54:15.773Z] 19:54:15 [task 2019-07-16T19:54:15.774Z] 19:54:15 [task 2019-07-16T19:54:15.791Z] 19:54:15 [task 2019-07-16T19:54:15.791Z] 19:54:15 [task 2019-07-16T19:54:15.794Z] 19:54:15 [task 2019-07-16T19:54:15.810Z] 19:54:15 [task 2019-07-16T19:54:15.847Z] 19:54:15 [task 2019-07-16T19:54:15.847Z] 19:54:15 [task 2019-07-16T19:54:15.847Z] 19:54:15 [task 2019-07-16T19:54:15.883Z] 19:54:15 [task 2019-07-16T19:54:15.883Z] 19:54:15 [task 2019-07-16T19:54:15.883Z] 19:54:15 [task 2019-07-16T19:54:15.886Z] 19:54:15 [task 2019-07-16T19:54:15.886Z] 19:54:15 [task 2019-07-16T19:54:15.886Z] 19:54:15 [task 2019-07-16T19:54:15.887Z] 19:54:15 [task 2019-07-16T19:54:15.888Z] 19:54:15 [task 2019-07-16T19:54:15.888Z] 19:54:15 [task 2019-07-16T19:54:15.888Z] 19:54:15 [task 2019-07-16T19:54:15.960Z] 19:54:15 [task 2019-07-16T19:54:15.960Z] 19:54:15 [task 2019-07-16T19:54:15.960Z] 19:54:15 [task 2019-07-16T19:54:15.960Z] 19:54:15 INFO - TEST-START | testing/firefox-ui/tests/functional/security/test_ssl_disabled_error_page.py TestSSLDisabledErrorPage.test_ssl_disabled_error_page
INFO - 1563306854835 Marionette DEBUG 11 <- [1,150,null,{"value":null}]
INFO - 1563306854845 Marionette DEBUG Accepted connection 12 from 127.0.0.1:48640
INFO - 1563306854847 Marionette DEBUG Closed connection 11
INFO - 1563306854851 Marionette DEBUG 12 -> [0,1,"WebDriver:NewSession",{"strictFileInteractability":true}]
INFO - 1563306854858 Marionette TRACE [23622320129] Frame script loaded
INFO - 1563306855154 Marionette TRACE [23622320129] Frame script registered
INFO - 1563306855210 Marionette DEBUG 12 <- [1,1,null,{"sessionId":"d62eea47-a5f0-4939-a8dc-6da98b4276d2","capabilities":{"browserName":"firefox","browserVersion":"70.0a ... ld/tmp9fbkDc.mozrunner","moz:shutdownTimeout":60000,"moz:useNonSpecCompliantPointerOrigin":false,"moz:webdriverClick":true}}]
INFO - 1563306855213 Marionette DEBUG 12 -> [0,2,"WebDriver:SetTimeouts",{"script":30000}]
INFO - 1563306855214 Marionette DEBUG 12 <- [1,2,null,{"value":null}]
INFO - 1563306855216 Marionette DEBUG 12 -> [0,3,"WebDriver:SetTimeouts",{"pageLoad":300000}]
INFO - 1563306855217 Marionette DEBUG 12 <- [1,3,null,{"value":null}]
INFO - 1563306855221 Marionette DEBUG 12 -> [0,4,"WebDriver:SetTimeouts",{"implicit":0}]
INFO - 1563306855224 Marionette DEBUG 12 <- [1,4,null,{"value":null}]
INFO - 1563306855227 Marionette DEBUG 12 -> [0,5,"WebDriver:GetWindowHandles",{}]
INFO - 1563306855228 Marionette DEBUG 12 <- [1,5,null,["23622320129"]]
INFO - 1563306855229 Marionette DEBUG 12 -> [0,6,"WebDriver:GetWindowHandles",{}]
INFO - 1563306855232 Marionette DEBUG 12 <- [1,6,null,["23622320129"]]
INFO - 1563306855234 Marionette DEBUG 12 -> [0,7,"Marionette:SetContext",{"value":"chrome"}]
INFO - 1563306855234 Marionette DEBUG 12 <- [1,7,null,{"value":null}]
INFO - 1563306855237 Marionette DEBUG 12 -> [0,8,"WebDriver:GetChromeWindowHandle",{}]
INFO - 1563306855346 Marionette DEBUG 12 -> [0,35,"WebDriver:Navigate",{"url":"about:about"}]
INFO - 1563306855358 Marionette TRACE [23622320129] Received DOM event beforeunload for https://extended-validation.badssl.com/
INFO - ++DOCSHELL 0x7fdeb7c49000 == 11 [pid = 841] [id = {42c18136-32e7-4719-b7c1-511ed2885f29}]
INFO - ++DOMWINDOW == 24 (0x7fdeb7eb5d40) [pid = 841] [serial = 36] [outer = (nil)]
INFO - ++DOMWINDOW == 25 (0x7fdeb934dc00) [pid = 841] [serial = 37] [outer = 0x7fdeb7eb5d40]
INFO - 1563306855398 Marionette TRACE [36] Frame script loaded
INFO - 1563306855399 Marionette TRACE [36] Frame script registered
INFO - [Parent 841, Main Thread] WARNING: Constructing RangeBoundary with invalid value: 'mRef || aOffset == 0', file /builds/worker/workspace/build/src/obj-firefox/dist/include/mozilla/RangeBoundary.h, line 79
INFO - [Parent 841, Main Thread] WARNING: Constructing RangeBoundary with invalid value: 'mRef || aOffset == 0', file /builds/worker/workspace/build/src/obj-firefox/dist/include/mozilla/RangeBoundary.h, line 79
INFO - [Parent 841, Main Thread] WARNING: Constructing RangeBoundary with invalid value: 'mRef || aOffset == 0', file /builds/worker/workspace/build/src/obj-firefox/dist/include/mozilla/RangeBoundary.h, line 79
INFO - [Parent 841, Main Thread] WARNING: Constructing RangeBoundary with invalid value: 'mRef || aOffset == 0', file /builds/worker/workspace/build/src/obj-firefox/dist/include/mozilla/RangeBoundary.h, line 79
INFO - 1563306855433 Marionette TRACE [23622320129] Received DOM event pagehide for https://extended-validation.badssl.com/
INFO - 1563306855439 Marionette TRACE [23622320129] Received DOM event unload for https://extended-validation.badssl.com/
INFO - 1563306855446 Marionette TRACE [23622320129] Received observer notification outer-window-destroyed
INFO - 1563306855475 Marionette TRACE [36] Check readyState uninitialized for about:blank
INFO - ++DOMWINDOW == 26 (0x7fdeb949dc00) [pid = 841] [serial = 38] [outer = 0x7fdeb7eb5d40]
INFO - [Parent 841, Main Thread] WARNING: Constructing RangeBoundary with invalid value: 'mRef || aOffset == 0', file /builds/worker/workspace/build/src/obj-firefox/dist/include/mozilla/RangeBoundary.h, line 79
INFO - [Parent 841, Main Thread] WARNING: Constructing RangeBoundary with invalid value: 'mRef || aOffset == 0', file /builds/worker/workspace/build/src/obj-firefox/dist/include/mozilla/RangeBoundary.h, line 79
INFO - [Parent 841, Main Thread] WARNING: Constructing RangeBoundary with invalid value: 'mRef || aOffset == 0', file /builds/worker/workspace/build/src/obj-firefox/dist/include/mozilla/RangeBoundary.h, line 79
INFO - [Parent 841, Main Thread] WARNING: Constructing RangeBoundary with invalid value: 'mRef || aOffset == 0', file /builds/worker/workspace/build/src/obj-firefox/dist/include/mozilla/RangeBoundary.h, line 79
INFO - 1563306855604 Marionette TRACE [36] Received DOM event DOMContentLoaded for about:about
INFO - 1563306855624 Marionette TRACE [36] Received DOM event pageshow for about:about
INFO - 1563306855702 Marionette DEBUG 12 <- [1,35,null,{"value":null}]
INFO - 1563306855708 Marionette DEBUG 12 -> [0,36,"Marionette:SetContext",{"value":"chrome"}]
INFO - 1563306855709 Marionette DEBUG 12 <- [1,36,null,{"value":null}]
INFO - 1563306855713 Marionette DEBUG 12 -> [0,37,"Marionette:GetContext",{}]
INFO - 1563306855714 Marionette DEBUG 12 <- [1,37,null,{"value":"chrome"}]
INFO - 1563306855716 Marionette DEBUG 12 -> [0,38,"Marionette:SetContext",{"value":"chrome"}]
INFO - 1563306855717 Marionette DEBUG 12 <- [1,38,null,{"value":null}]
INFO - 1563306855722 Marionette DEBUG 12 -> [0,39,"WebDriver:ExecuteAsyncScript",{"scriptTimeout":null,"newSandbox":true,"args":[{"sessions":true}],"filename":"../../ven ... lve(true);\n }, aError => {\n resolve(false);\n });","sandbox":"default","line":75}]
INFO - --DOCSHELL 0x7f1a99bbd000 == 0 [pid = 1190] [id = {05e51b47-2d5f-44fb-a250-28d9e845ef40}] [url = https://extended-validation.badssl.com/]
INFO - Assertion failure: false (Active connection not found), at /builds/worker/workspace/build/src/netwerk/protocol/http/nsHttpConnectionMgr.cpp:2854
INFO - #01: ???[/builds/worker/workspace/build/application/firefox/libxul.so +0x11aae4f]
INFO - #02: ???[/builds/worker/workspace/build/application/firefox/libxul.so +0xd5f35e]
INFO - #03: ???[/builds/worker/workspace/build/application/firefox/libxul.so +0xd61899]
INFO - #04: ???[/builds/worker/workspace/build/application/firefox/libxul.so +0xe96b5c]
INFO - #05: ???[/builds/worker/workspace/build/application/firefox/libxul.so +0xe975af]
INFO - #06: ???[/builds/worker/workspace/build/application/firefox/libxul.so +0xd5f35e]
INFO - #07: ???[/builds/worker/workspace/build/application/firefox/libxul.so +0xd61899]
INFO - #08: ???[/builds/worker/workspace/build/application/firefox/libxul.so +0x12b5f07]
INFO - #09: ???[/builds/worker/workspace/build/application/firefox/libxul.so +0x126b2ed]
INFO - #10: ???[/builds/worker/workspace/build/application/firefox/libxul.so +0x126b248]
INFO - #11: ???[/builds/worker/workspace/build/application/firefox/libxul.so +0xd5cae2]
INFO - #12: ???[/builds/worker/workspace/build/application/firefox/libnspr4.so +0x2b388]
INFO - #13: ???[/lib/x86_64-linux-gnu/libpthread.so.0 +0x76ba]
INFO - #14: clone[/lib/x86_64-linux-gnu/libc.so.6 +0x10741d]
INFO - #15: ??? (???:???)
INFO - ExceptionHandler::GenerateDump cloned child 1235
INFO - ExceptionHandler::SendContinueSignalToChild sent continue signal to child
INFO - ExceptionHandler::WaitForContinueSignal waiting for continue signal...
INFO - 1563306855801 Marionette DEBUG 12 <- [1,39,null,{"value":true}]
INFO - --DOMWINDOW == 2 (0x7f1a9abdfd40) [pid = 1190] [serial = 1] [outer = (nil)] [url = https://extended-validation.badssl.com/]
INFO - --DOMWINDOW == 1 (0x7f1a9a1bf800) [pid = 1190] [serial = 2] [outer = (nil)] [url = about:blank]
INFO - --DOMWINDOW == 0 (0x7f1a9a1ce400) [pid = 1190] [serial = 3] [outer = (nil)] [url = https://extended-validation.badssl.com/]
INFO - [Child 1190, Main Thread] WARNING: XPCOM objects created/destroyed from static ctor/dtor: file /builds/worker/workspace/build/src/xpcom/base/nsTraceRefcnt.cpp, line 194
INFO - [Child 1190, Main Thread] WARNING: XPCOM objects created/destroyed from static ctor/dtor: file /builds/worker/workspace/build/src/xpcom/base/nsTraceRefcnt.cpp, line 194
INFO - nsStringStats
INFO - => mAllocCount: 9322
INFO - => mReallocCount: 0
INFO - => mFreeCount: 9322
INFO - => mShareCount: 6970
INFO - => mAdoptCount: 651
INFO - => mAdoptFreeCount: 655
INFO - => Process ID: 1190, Thread ID: 139752668866368
INFO - Exiting due to channel error.
INFO - [GFX1-]: Receive IPC close with reason=AbnormalShutdown
INFO - [Child 899, Chrome_ChildThread] WARNING: pipe error (3): Connection reset by peer: file /builds/worker/workspace/build/src/ipc/chromium/src/chrome/common/ipc_channel_posix.cc, line 358
INFO - Exiting due to channel error.

Flags: needinfo?(honzab.moz)

Liz Henry (:lizzard Please n-i to RyanVM, jcristau, or pascal)

Updated

•

5 years ago

Whiteboard: [necko-triaged][secure-proxy-skyline][http-conn] → [necko-triaged][secure-proxy][skyline][http-conn]

Pascal Chevrel:pascalc (PTO until August 21)

Updated

•

5 years ago

status-firefox68: --- → wontfix

status-firefox69: --- → affected

status-firefox-esr60: --- → wontfix

status-firefox-esr68: --- → affected

Honza Bambas (:mayhemer)

Assignee

Comment 6

•

5 years ago

https://treeherder.mozilla.org/#/jobs?repo=try&revision=25ac8846a4d63ef3a0eb8b26be218684a5dd3a28

Flags: needinfo?(honzab.moz)

Pulsebot

Comment 7

•

5 years ago

Pushed by honzab.moz@firemni.cz: https://hg.mozilla.org/integration/autoland/rev/687930978a2a Remove HTTP/2 connection (e2e or proxy) from the opposite (an)onymous connection entry in case it cannot be found in the entry the connection is associated with because of HTTP/2 (an)onymous connection pool sharing, r=kershaw

Oana Pop-Rus

Comment 8

•

5 years ago

bugherder

https://hg.mozilla.org/mozilla-central/rev/687930978a2a

Status: ASSIGNED → RESOLVED

Closed: 5 years ago

status-firefox70: affected → fixed

Resolution: --- → FIXED

Target Milestone: --- → mozilla70

Honza Bambas (:mayhemer)

Assignee

Updated

•

5 years ago

Regressions: 1571405

Ritu Kothari (:ritu) (Inactive, please n-i to RyanVM, jcristau, or pascal)

Comment 9

•

5 years ago

Hi Honza, since 69 is marked as affected, could you please nominate an uplift of this fix to beta69? Thanks!

Flags: needinfo?(honzab.moz)

Honza Bambas (:mayhemer)

Assignee

Comment 10

•

5 years ago

Comment on attachment 9077707 [details]
Bug 1565242 - Remove HTTP/2 connection (e2e or proxy) from the opposite (an)onymous connection entry in case it cannot be found in the entry the connection is associated with because of HTTP/2 (an)onymous connection pool sharing, r=kershaw!

Beta/Release Uplift Approval Request

User impact if declined: indefinite connection hangs
Is this code covered by automated tests?: No
Has the fix been verified in Nightly?: Yes
Needs manual test from QE?: Yes
If yes, steps to reproduce: https://bugzilla.mozilla.org/show_bug.cgi?id=1565242#c1
List of other uplifts needed: None
Risk to taking this patch: Low
Why is the change risky/not risky? (and alternatives if risky): just releases an unused connection and thus drops the active connection counter appropriately, I don't expect any problems here
String changes made/needed:

Flags: needinfo?(honzab.moz)

Attachment #9077707 - Flags: approval-mozilla-beta?

Honza Bambas (:mayhemer)

Assignee

Updated

•

5 years ago

Flags: qe-verify+

Brindusa Tot, Desktop QA

Updated

•

5 years ago

QA Whiteboard: [qa-triaged]

Pascal Chevrel:pascalc (PTO until August 21)

Comment 11

•

5 years ago

Comment on attachment 9077707 [details]
Bug 1565242 - Remove HTTP/2 connection (e2e or proxy) from the opposite (an)onymous connection entry in case it cannot be found in the entry the connection is associated with because of HTTP/2 (an)onymous connection pool sharing, r=kershaw!

Low risk and affects skyline, uplift approved for 69 beta 12, thanks.

Attachment #9077707 - Flags: approval-mozilla-beta? → approval-mozilla-beta+

Stefan Hindli [:stefan_hindli]

Comment 12

•

5 years ago

bugherder uplift

https://hg.mozilla.org/releases/mozilla-beta/rev/7b3d9986def6

status-firefox69: affected → fixed

Peter Magyari (Desktop QA)

Comment 13

•

5 years ago

Verified fixed on Firefox Beta 69.0b9(20190730004747) and Firefox Nightly 70.0a1(20190807215212) using Windows 10, MacOS 10.13.6 and Ubuntu 18.04.

Status: RESOLVED → VERIFIED

status-firefox69: fixed → verified

status-firefox70: fixed → verified

Flags: qe-verify+

Ryan VanderMeulen [:RyanVM]

Comment 14

•

5 years ago

Is this something we need on ESR68 or can we wontfix it?

Flags: needinfo?(honzab.moz)

Honza Bambas (:mayhemer)

Assignee

Comment 15

•

5 years ago

(In reply to Ryan VanderMeulen [:RyanVM] from comment #14)

Is this something we need on ESR68 or can we wontfix it?

This mostly affects only h2 proxying under some specific conditions. I think this change is too sensitive to uplift just like that w/o baking it first.

Flags: needinfo?(honzab.moz)

Ryan VanderMeulen [:RyanVM]

Updated

•

5 years ago

status-firefox-esr68: affected → wontfix

BugBot [:suhaib / :marco/ :calixte]

Updated

•

3 years ago

Has Regression Range: --- → yes