nagai/fune
3
0
Fork 0
forked from mirrors/gecko-dev
Code Pull requests Activity

Bug 1823417 - Don't exclude Http3 origin for WebTransport, r=jesup,necko-reviewers

Browse source 
Differential Revision: https://phabricator.services.mozilla.com/D173039
This commit is contained in:
Kershaw Chang 2023-03-23 09:38:33 +00:00
parent 32434f8e6b
commit 6c5beed2e2
1 changed files with 6 additions and 3 deletions
repo.diff.show_diff_stats Download patch file Download diff file Expand all files Collapse all files

9
netwerk/protocol/http/Http3Session.cpp
View file

@ -229,7 +229,7 @@ void Http3Session::Shutdown() {
(mError == NS_ERROR_NET_HTTP3_PROTOCOL_ERROR)) &&
(mError !=
mozilla::psm::GetXPCOMFromNSSError(SSL_ERROR_BAD_CERT_DOMAIN)) &&
!isEchRetry) {
!isEchRetry && !mConnInfo->GetWebTransport()) {
gHttpHandler->ExcludeHttp3(mConnInfo);
}
@ -1730,9 +1730,12 @@ void Http3Session::CloseStreamInternal(Http3StreamBase* aStream,
}
mWebTransportSessions.RemoveElement(aStream);
mWebTransportStreams.RemoveElement(aStream);
// Close(NS_OK) implies that the NeqoHttp3Conn will be closed, so we can only
// do this when there is no Http3Steeam, WebTransportSession and
// WebTransportStream.
if ((mShouldClose || mGoawayReceived) &&
(!mStreamTransactionHash.Count() || !mWebTransportSessions.IsEmpty() ||
!mWebTransportStreams.IsEmpty())) {
(!mStreamTransactionHash.Count() && mWebTransportSessions.IsEmpty() &&
mWebTransportStreams.IsEmpty())) {
MOZ_ASSERT(!IsClosing());
Close(NS_OK);
}