Merge pull request bigbluebutton#19916 from gustavotrott/test-graphql…

…-reconnecting refactor (graphql-middleware): Increase Ws Read Limit to 10MB (necessary for annotations)
abc1225 · Mar 27, 2024 · 638a327 · 638a327
2 parents f684b7d + 6fca6c2
commit 638a327
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 2 deletions.
diff --git a/bbb-graphql-middleware/internal/hascli/conn/writer/writer.go b/bbb-graphql-middleware/internal/hascli/conn/writer/writer.go
@@ -1,6 +1,8 @@
 package writer
 
 import (
+	"context"
+	"errors"
 	"github.com/iMDT/bbb-graphql-middleware/internal/common"
 	"github.com/iMDT/bbb-graphql-middleware/internal/msgpatch"
 	log "github.com/sirupsen/logrus"
@@ -144,7 +146,9 @@ RangeLoop:
 				log.Tracef("sending to hasura: %v", fromBrowserMessageAsMap)
 				err := wsjson.Write(hc.Context, hc.Websocket, fromBrowserMessageAsMap)
 				if err != nil {
-					log.Errorf("error on write (we're disconnected from hasura): %v", err)
+					if !errors.Is(err, context.Canceled) {
+						log.Errorf("error on write (we're disconnected from hasura): %v", err)
+					}
 					return
 				}
 			}

diff --git a/bbb-graphql-middleware/internal/websrv/connhandler.go b/bbb-graphql-middleware/internal/websrv/connhandler.go
@@ -48,7 +48,7 @@ func ConnectionHandler(w http.ResponseWriter, r *http.Request) {
 	}
 
 	browserWsConn, err := websocket.Accept(w, r, &acceptOptions)
-	browserWsConn.SetReadLimit(999999)
+	browserWsConn.SetReadLimit(9999999) //10MB
 	if err != nil {
 		log.Errorf("error: %v", err)
 	}