Merge pull request #325 from dwelch2344/swallow-ni-exception

rose-a · web-flow · commit dd418d16b690 · 2021-03-02T21:01:16.000+01:00
Catch NotImplementedException #324
diff --git a/src/GraphQL.Client/Websocket/GraphQLHttpWebSocket.cs b/src/GraphQL.Client/Websocket/GraphQLHttpWebSocket.cs
@@ -408,6 +408,10 @@ public Task InitializeWebSocket()
                 {
                     _clientWebSocket.Options.ClientCertificates = ((HttpClientHandler)Options.HttpMessageHandler).ClientCertificates;
                 }
+                catch (NotImplementedException)
+                {
+                    Debug.WriteLine("property 'ClientWebSocketOptions.ClientCertificates' not implemented by current platform");                
+                }
                 catch (PlatformNotSupportedException)
                 {
                     Debug.WriteLine("property 'ClientWebSocketOptions.ClientCertificates' not supported by current platform");
@@ -417,6 +421,10 @@ public Task InitializeWebSocket()
                 {
                     _clientWebSocket.Options.UseDefaultCredentials = ((HttpClientHandler)Options.HttpMessageHandler).UseDefaultCredentials;
                 }
+                catch (NotImplementedException)
+                {
+                    Debug.WriteLine("property 'ClientWebSocketOptions.UseDefaultCredentials' not implemented by current platform");    
+                }
                 catch (PlatformNotSupportedException)
                 {
                     Debug.WriteLine("Property 'ClientWebSocketOptions.UseDefaultCredentials' not supported by current platform");

Original file line number	Diff line number	Diff line change
`@@ -408,6 +408,10 @@ public Task InitializeWebSocket()`
`408`	`408`	`{`
`409`	`409`	`_clientWebSocket.Options.ClientCertificates = ((HttpClientHandler)Options.HttpMessageHandler).ClientCertificates;`
`410`	`410`	`}`
	`411`	`+ catch (NotImplementedException)`
	`412`	`+ {`
	`413`	`+ Debug.WriteLine("property 'ClientWebSocketOptions.ClientCertificates' not implemented by current platform");`
	`414`	`+ }`
`411`	`415`	`catch (PlatformNotSupportedException)`
`412`	`416`	`{`
`413`	`417`	`Debug.WriteLine("property 'ClientWebSocketOptions.ClientCertificates' not supported by current platform");`
`@@ -417,6 +421,10 @@ public Task InitializeWebSocket()`
`417`	`421`	`{`
`418`	`422`	`_clientWebSocket.Options.UseDefaultCredentials = ((HttpClientHandler)Options.HttpMessageHandler).UseDefaultCredentials;`
`419`	`423`	`}`
	`424`	`+ catch (NotImplementedException)`
	`425`	`+ {`
	`426`	`+ Debug.WriteLine("property 'ClientWebSocketOptions.UseDefaultCredentials' not implemented by current platform");`
	`427`	`+ }`
`420`	`428`	`catch (PlatformNotSupportedException)`
`421`	`429`	`{`
`422`	`430`	`Debug.WriteLine("Property 'ClientWebSocketOptions.UseDefaultCredentials' not supported by current platform");`