use bodyclose linter and fix errors it reports (#389)

.golangci.yml

@@ -15,6 +15,7 @@ linters-settings:
 
 linters:
   enable:
+    - bodyclose
     - gocritic
     - gocyclo
     - gofmt

x/ref/runtime/protocols/lib/websocket/ws.go

@@ -12,12 +12,10 @@ import (
 	"net/url"
 	"time"
 
-	"v.io/x/ref/runtime/protocols/lib/tcputil"
-
 	"github.com/gorilla/websocket"
-
 	"v.io/v23/context"
 	"v.io/v23/flow"
+	"v.io/x/ref/runtime/protocols/lib/tcputil"
 )
 
 // TODO(jhahn): Figure out a way for this mapping to be shared.
@@ -46,11 +44,18 @@ func (WS) Dial(ctx *context.T, protocol, address string, timeout time.Duration)
 		return nil, err
 	}
 
-	//nolint:staticcheck //lint:ignore SA1019
-	ws, _, err := websocket.NewClient(conn, u, http.Header{}, bufferSize, bufferSize)
+	dialer := &websocket.Dialer{
+		ReadBufferSize:  bufferSize,
+		WriteBufferSize: bufferSize,
+		NetDial: func(net, addr string) (net.Conn, error) {
+			return conn, nil
+		},
+	}
+	ws, resp, err := dialer.DialContext(ctx, u.String(), http.Header{})
 	if err != nil {
 		return nil, err
 	}
+	resp.Body.Close()
 	var zero time.Time
 	conn.SetDeadline(zero) //nolint:errcheck
 	return WebsocketConn(ws), nil

x/ref/services/internal/pproflib/proxy_test.go

@@ -60,6 +60,7 @@ func TestPProfProxy(t *testing.T) {
 		if err == nil && resp.StatusCode == 200 {
 			break
 		}
+		resp.Body.Close()
 		time.Sleep(time.Second)
 	}