Skip to content

Commit 98893bf

Browse files
authored
Update HttpConnection.cs
1 parent 59e89bd commit 98893bf

File tree

1 file changed

+12
-21
lines changed

1 file changed

+12
-21
lines changed

src/SignalR/clients/csharp/Http.Connections.Client/src/HttpConnection.cs

Lines changed: 12 additions & 21 deletions
Original file line numberDiff line numberDiff line change
@@ -539,27 +539,18 @@ private HttpClient CreateHttpClient()
539539
httpClientHandler.Proxy = _httpConnectionOptions.Proxy;
540540
}
541541

542-
try
543-
{
544-
// On supported platforms, we need to pass the cookie container to the http client
545-
// so that we can capture any cookies from the negotiate response and give them to WebSockets.
546-
httpClientHandler.CookieContainer = _httpConnectionOptions.Cookies;
547-
}
548-
// Some variants of Mono do not support client certs or cookies and will throw NotImplementedException or NotSupportedException
549-
// Also WASM doesn't support some settings in the browser
550-
catch (Exception ex) when (ex is NotSupportedException || ex is NotImplementedException)
551-
{
552-
Log.CookiesNotSupported(_logger);
553-
}
554-
555-
// Only access HttpClientHandler.ClientCertificates
556-
// if the user has configured those options
557-
// https://github.com/aspnet/SignalR/issues/2232
558-
var clientCertificates = _httpConnectionOptions.ClientCertificates;
559-
if (clientCertificates?.Count > 0)
560-
{
561-
httpClientHandler.ClientCertificates.AddRange(clientCertificates);
562-
}
542+
try
543+
{
544+
// On supported platforms, we need to pass the cookie container to the http client
545+
// so that we can capture any cookies from the negotiate response and give them to WebSockets.
546+
httpClientHandler.CookieContainer = _httpConnectionOptions.Cookies;
547+
}
548+
// Some variants of Mono do not support client certs or cookies and will throw NotImplementedException or NotSupportedException
549+
// Also WASM doesn't support some settings in the browser
550+
catch (Exception ex) when (ex is NotSupportedException || ex is NotImplementedException)
551+
{
552+
Log.CookiesNotSupported(_logger);
553+
}
563554

564555
// Only access HttpClientHandler.ClientCertificates
565556
// if the user has configured those options

0 commit comments

Comments
 (0)