Fix bug classifying P2P failure reason

Just because we did not gather any reflexive address that was enabled, that
we failed to STUN.

"Failed to gather any" is ambiguous English.  We really meant "we did not gather
any".

Also, if we have a public IPv4 address, it could be that we did not STUN,
because the STUN server told us that our address was already public, and so
was not gathered as an additional address.
This commit is contained in:
Fletcher Dunn
2020-07-03 15:35:15 -07:00
parent 9b52cfd975
commit 9dd1e35a05
@@ -432,7 +432,7 @@ void CSteamNetworkConnectionP2P::GuessICEFailureReason( ESteamNetConnectionEnd &
}
// We failed to STUN?
if ( nFailedToGatherTypes & k_EICECandidate_Any_Reflexive )
if ( ( nAllowedTypes & k_EICECandidate_Any_Reflexive ) != 0 && ( nGatheredTypes & (k_EICECandidate_Any_Reflexive|k_EICECandidate_IPv4_HostPublic) ) == 0 )
{
if ( m_connectionConfig.m_P2P_STUN_ServerList.Get().empty() )
{