workaround for webkit websocket compression bug (#45)

getlantern · Dec 2, 2022 · adb7c82 · adb7c82
1 parent 0f60fc8
commit adb7c82
Showing 1 changed file with 11 additions and 1 deletion.
diff --git a/egress/egress.go b/egress/egress.go
@@ -65,7 +65,17 @@ func (l proxyListener) Addr() net.Addr {
 func (l proxyListener) handleWebsocket(w http.ResponseWriter, r *http.Request) {
 	// TODO: InsecureSkipVerify=true just disables origin checking, we need to instead add origin
 	// patterns as strings using AcceptOptions.OriginPattern
-	c, err := websocket.Accept(w, r, &websocket.AcceptOptions{InsecureSkipVerify: true})
+	// TODO: disabling compression is a workaround for a WebKit bug:
+	// https://github.com/getlantern/broflake/issues/45
+	c, err := websocket.Accept(
+		w,
+		r,
+		&websocket.AcceptOptions{
+			InsecureSkipVerify: true,
+			CompressionMode:    websocket.CompressionDisabled,
+		},
+	)
+
 	if err != nil {
 		// TODO: this is the idiom for our WebSocket library, but we should log the err better
 		fmt.Println(err)