fix: only use WebTransport when TransportWebTransports is set as clie…

…nt option
philippseith · Jan 8, 2024 · 6aa9891 · 6aa9891
1 parent 4a9ea04
commit 6aa9891
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/httpconnection.go b/httpconnection.go
@@ -3,7 +3,6 @@ package signalr
 import (
 	"context"
 	"encoding/json"
-	"errors"
 	"fmt"
 	"io"
 	"net/http"
@@ -127,7 +126,7 @@ func NewHTTPConnection(ctx context.Context, address string, options ...func(*htt
 	// Select the best connection
 	var conn Connection
 	switch {
-	case negotiateResponse.hasTransport("WebTransports"):
+	case httpConn.hasTransport(TransportWebTransports) && negotiateResponse.hasTransport(TransportWebTransports):
 		var d webtransport.Dialer
 		_, wtConn, err := d.Dial(ctx, reqURL.String(), req.Header)
 		if err != nil {

diff --git a/negotiateresponse.go b/negotiateresponse.go
@@ -3,6 +3,7 @@ package signalr
 type TransportType string
 
 var TransportWebSockets TransportType = "WebSockets"
+var TransportWebTransports TransportType = "WebTransports"
 var TransportServerSentEvents TransportType = "ServerSentEvents"
 
 type TransferFormatType string