Bug 1667351 - Process neqo events immediately after a cert authentication is done. r=necko-reviewers,valentin

Currently we call ResumeSend() which will not process neqo events. It will only produce new packets. We are relying on other events, like timeout or a receive packet to trigger this.

Differential Revision: https://phabricator.services.mozilla.com/D92113
This commit is contained in:
Dragana Damjanovic 2020-10-05 19:03:55 +00:00
parent 63230729cd
commit 030fcd3a0e

View File

@ -1582,10 +1582,13 @@ void Http3Session::Authenticated(int32_t aError) {
static_cast<uint32_t>(mError), this));
}
mHttp3Connection->PeerAuthenticated(aError);
}
if (mConnection) {
Unused << mConnection->ResumeSend();
// Call OnQuicTimeoutExpired to properly process neqo events and outputs.
// We call OnQuicTimeoutExpired instead of ProcessOutputAndEvents, because
// HttpConnectionUDP must close this session in case of an error.
NS_DispatchToCurrentThread(NewRunnableMethod(
"net::HttpConnectionUDP::OnQuicTimeoutExpired", mSegmentReaderWriter,
&HttpConnectionUDP::OnQuicTimeoutExpired));
}
}