1996076 - Pass NS_HTTP_LARGE_KEEPALIVE to neqo

Status: NEW

(Core :: Networking, task, P3)

Description

[Max Inden]

Firefox has NS_HTTP_LARGE_KEEPALIVE:

https://github.com/mozilla-firefox/firefox/blob/48f8a74041be2973d7df8ded3ad5e64c2c00e8dd/netwerk/protocol/http/nsHttp.h#L86

This is e.g. set for DOH connections:

https://github.com/mozilla-firefox/firefox/blob/48f8a74041be2973d7df8ded3ad5e64c2c00e8dd/netwerk/protocol/http/TRRServiceChannel.cpp#L493-L496

But not passed to HTTP/3 connections:

https://github.com/mozilla-firefox/firefox/blob/48f8a74041be2973d7df8ded3ad5e64c2c00e8dd/netwerk/protocol/http/Http3Session.cpp#L160-L163

But instead for HTTP/3 we depend on a separate pref:

https://github.com/mozilla-firefox/firefox/blob/48f8a74041be2973d7df8ded3ad5e64c2c00e8dd/modules/libpref/init/StaticPrefList.yaml#L14930-L14938

@Valentin, let me know if this documents our past conversation well enough. I don't think this is high priority.

Comment 1

[Valentin Gosu [:valentin] {{ PTO 3 July - 3 August }}]

I think that sounds correct. Doesn't seem to be a big issue at the moment, but it would be nice if h2 and h3 handled NS_HTTP_LARGE_KEEPALIVE the same. 🤷