1655895 - bogus timeout passed to Http3Session::SetupTimer

Status: RESOLVED FIXED

(Core :: Networking: HTTP, defect, P2)

Description

[Junior [inactive]]

I often see the bogus log
Http3Session::SetupTimer to 18446744073709551615 ms

Here's the culprit:
Output::None passed uint::max to indicated a no-op.
We should not set a timer with uint64::max as a delay which hurts performance

Comment 1

[Junior [inactive]]

Attachment: Bug 1655895 - do not set a timer for neqo passed a UINT64::MAX, r=dragana

Depends on D85233

Comment 2

[BugBot [:suhaib / :marco/ :calixte]]

There's a r+ patch which didn't land and no activity in this bug for 2 weeks.
:CuveeHsu, could you have a look please?
For more information, please visit auto_nag documentation.

Comment 3

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/0f723de1af73
do not set a timer for neqo passed a UINT64::MAX, r=dragana,necko-reviewers

Comment 4

[Noemi Erli[:noemi_erli]]

https://hg.mozilla.org/mozilla-central/rev/0f723de1af73

Comment 5

[Junior [inactive]]

Thanks, Dragana.