fix: superfluous response.WriteHeader call

Signed-off-by: Christian Stewart <christian@aperture.us>
aperturerobotics · Jan 2, 2025 · 631c978 · 631c978
1 parent 17988ae
commit 631c978
Showing 1 changed file with 3 additions and 2 deletions.
diff --git a/srpc/server-http.go b/srpc/server-http.go
@@ -36,8 +36,9 @@ func (s *HTTPServer) ServeHTTP(w http.ResponseWriter, r *http.Request) {
 
 	c, err := websocket.Accept(w, r, &websocket.AcceptOptions{})
 	if err != nil {
-		w.WriteHeader(500)
-		_, _ = w.Write([]byte(err.Error() + "\n"))
+		// NOTE: the error is already written with http.Error
+		// w.WriteHeader(500)
+		// _, _ = w.Write([]byte(err.Error() + "\n"))
 		return
 	}
 	defer c.Close(websocket.StatusInternalError, "closed")