Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
Fix System.Net.Sockets telemetry (#42188)
* Correct NameResolutionTelemetry logic Taken from c4c9a2f99b7e339388199086d3014041abccc21e * Enable listening to multiple sources with TestEventListener * Workaround EventWrittenEventArgs bug when the EventArgs are stored Workaround #42128 * Correct System.Net.Sockets Telemetry * Avoid using value tuple in TestEventListener * Remove unnecessary argument to OnCompletedInternal * Remove redundant Telemetry.IsEnabled check * Log Connect/Accept start before the initial context capture * Use SocketHelperBase in Accept tests * Avoid duplicate events for BeginConnect without ConnextEx support * Enable Sync Socket tests * Revert unrelated SocketPal change * Log the correct ErrorCode in case of socket disposal * Add more info on TelemetryTest timeout * Add PlatformSpecific attribute to ConnectFailure test * Add missing BeginConnect AfterConnect call on sync failure * Add comment around GetHelperBase * Correct WaitForEventCountersAsync helper * Assert against SocketError.TimedOut in ConnectFailed test * Add EndConnect comment * Log Failure around sync SocketPal exceptions * Don't assert that the exception message is empty * Dispose socket in a different Thread * Disable Telemetry failure tests for Sync on RedHat7 * Use more descriptive names in MemberData generation * Avoid using reflection for #42128 workaround * Remove ConnectCanceled event
- Loading branch information