Bug 1756709 Comment 3 Edit History

Note: The actual edited comment in the bug view page will always show the original commenter’s name and original timestamp.

Original comment by
on 
It turns out that an `AltSvcTransaction` supposes not have [NS_HTTP_ALLOW_KEEPALIVE](https://searchfox.org/mozilla-central/rev/8f42809e51cb07aa4f5739932a06d14581e9dd4a/netwerk/protocol/http/AlternateServices.cpp#509) flag. Look like the purpose of this is to bypass the [RestrictConnection check](https://searchfox.org/mozilla-central/rev/8f42809e51cb07aa4f5739932a06d14581e9dd4a/netwerk/protocol/http/nsHttpConnectionMgr.cpp#3355).
However, in the [constructor of NullHTTPTransaction](https://searchfox.org/mozilla-central/rev/8f42809e51cb07aa4f5739932a06d14581e9dd4a/netwerk/protocol/http/NullHttpTransaction.cpp#29), `NS_HTTP_ALLOW_KEEPALIVE` is set again.
I think we need to fix this.
Revision 1 by
on 
It turns out that an `AltSvcTransaction` supposes not to have [NS_HTTP_ALLOW_KEEPALIVE](https://searchfox.org/mozilla-central/rev/8f42809e51cb07aa4f5739932a06d14581e9dd4a/netwerk/protocol/http/AlternateServices.cpp#509) flag. Looks like the purpose of this is to bypass the [RestrictConnection check](https://searchfox.org/mozilla-central/rev/8f42809e51cb07aa4f5739932a06d14581e9dd4a/netwerk/protocol/http/nsHttpConnectionMgr.cpp#3355).
However, in the [constructor of NullHTTPTransaction](https://searchfox.org/mozilla-central/rev/8f42809e51cb07aa4f5739932a06d14581e9dd4a/netwerk/protocol/http/NullHttpTransaction.cpp#29), `NS_HTTP_ALLOW_KEEPALIVE` is set again.
I think we need to fix this.

Back to Bug 1756709 Comment 3