NatsWebSocketOpts improvements (#610)

* Fence added, an occurs exceptions when add headers to blazor
nats-io · Sep 8, 2024 · e6b9d1c · e6b9d1c
1 parent 4da55d4
commit e6b9d1c
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 2 deletions.
diff --git a/src/NATS.Client.Core/Internal/DotnetRuntimeConstants.cs b/src/NATS.Client.Core/Internal/DotnetRuntimeConstants.cs
@@ -0,0 +1,8 @@
+using System.Runtime.InteropServices;
+
+namespace NATS.Client.Core.Internal;
+
+internal static class DotnetRuntimeConstants
+{
+    public static readonly OSPlatform BrowserPlatform = OSPlatform.Create("Browser");
+}
diff --git a/src/NATS.Client.Core/NatsWebSocketOpts.cs b/src/NATS.Client.Core/NatsWebSocketOpts.cs
@@ -1,4 +1,5 @@
 using System.Net.WebSockets;
+using System.Runtime.InteropServices;
 using Microsoft.Extensions.Primitives;
 using NATS.Client.Core.Internal;
 
@@ -32,8 +33,7 @@ internal async ValueTask ApplyClientWebSocketOptionsAsync(
         NatsTlsOpts tlsOpts,
         CancellationToken cancellationToken)
     {
-        // todo: test that this doesn't throw in Blazor, otherwise we need a way to detect Blazor and skip
-        if (RequestHeaders != null)
+        if (RequestHeaders != null && !RuntimeInformation.IsOSPlatform(DotnetRuntimeConstants.BrowserPlatform))
         {
             foreach (var entry in RequestHeaders)
             {