Fix ArgumentNullException from ClientWebSocketOptions.ClientCertificates (#431)

sungam3r · web-flow · commit 8379f8f2fc66 · 2022-08-01T23:12:33.000+02:00
diff --git a/src/GraphQL.Client/Websocket/GraphQLHttpWebSocket.cs b/src/GraphQL.Client/Websocket/GraphQLHttpWebSocket.cs
@@ -399,7 +399,9 @@ public Task InitializeWebSocket()
             // the following properties are not supported in Blazor WebAssembly and throw a PlatformNotSupportedException error when accessed
             try
             {
-                _clientWebSocket.Options.ClientCertificates = ((HttpClientHandler)Options.HttpMessageHandler).ClientCertificates;
+                var certs = ((HttpClientHandler)Options.HttpMessageHandler).ClientCertificates;
+                if (certs != null) // ClientWebSocketOptions.ClientCertificates.set throws ArgumentNullException
+                    _clientWebSocket.Options.ClientCertificates = certs;
             }
             catch (NotImplementedException)
             {

Original file line number	Diff line number	Diff line change
`@@ -399,7 +399,9 @@ public Task InitializeWebSocket()`
`399`	`399`	`// the following properties are not supported in Blazor WebAssembly and throw a PlatformNotSupportedException error when accessed`
`400`	`400`	`try`
`401`	`401`	`{`
`402`		`- _clientWebSocket.Options.ClientCertificates = ((HttpClientHandler)Options.HttpMessageHandler).ClientCertificates;`
	`402`	`+ var certs = ((HttpClientHandler)Options.HttpMessageHandler).ClientCertificates;`
	`403`	`+ if (certs != null) // ClientWebSocketOptions.ClientCertificates.set throws ArgumentNullException`
	`404`	`+ _clientWebSocket.Options.ClientCertificates = certs;`
`403`	`405`	`}`
`404`	`406`	`catch (NotImplementedException)`
`405`	`407`	`{`