starpc 0.13.0 → 0.13.2

package/e2e/e2e_test.go

@@ -30,7 +30,7 @@ func RunE2E(t *testing.T, cb func(client echo.SRPCEchoerClient) error) {
 	clientPipe, serverPipe := net.Pipe()
 
 	// outbound=true
-	clientMp, err := srpc.NewMuxedConn(clientPipe, true)
+	clientMp, err := srpc.NewMuxedConn(clientPipe, true, nil)
 	if err != nil {
 		t.Fatal(err.Error())
 	}
@@ -38,7 +38,7 @@ func RunE2E(t *testing.T, cb func(client echo.SRPCEchoerClient) error) {
 
 	ctx := context.Background()
 	// outbound=false
-	serverMp, err := srpc.NewMuxedConn(serverPipe, false)
+	serverMp, err := srpc.NewMuxedConn(serverPipe, false, nil)
 	if err != nil {
 		t.Fatal(err.Error())
 	}

package/echo/echo_srpc.pb.go

@@ -1,5 +1,5 @@
 // Code generated by protoc-gen-srpc. DO NOT EDIT.
-// protoc-gen-srpc version: v0.12.3-0.20221016080028-3e0cb248fdee
+// protoc-gen-srpc version: v0.13.2
 // source: github.com/aperturerobotics/starpc/echo/echo.proto
 
 package echo

package/echo/server.go

@@ -84,7 +84,7 @@ func (s *EchoServer) EchoBidiStream(strm SRPCEchoer_EchoBidiStreamStream) error
 
 // RpcStream runs a rpc stream
 func (r *EchoServer) RpcStream(stream SRPCEchoer_RpcStreamStream) error {
-	return rpcstream.HandleRpcStream(stream, func(ctx context.Context, componentID string) (srpc.Mux, func(), error) {
+	return rpcstream.HandleRpcStream(stream, func(ctx context.Context, componentID string) (srpc.Invoker, func(), error) {
 		if r.rpcStreamMux == nil {
 			return nil, nil, errors.New("not implemented")
 		}

package/go.mod

@@ -34,7 +34,7 @@ require (
 	github.com/multiformats/go-varint v0.0.6 // indirect
 	github.com/spacemonkeygo/spacelog v0.0.0-20180420211403-2296661a0572 // indirect
 	github.com/spaolacci/murmur3 v1.1.0 // indirect
-	golang.org/x/crypto v0.0.0-20220525230936-793ad666bf5e // indirect
-	golang.org/x/sys v0.0.0-20220811171246-fbc7d0a398ab // indirect
+	golang.org/x/crypto v0.1.1-0.20221024173537-a3485e174077 // indirect
+	golang.org/x/sys v0.1.1-0.20221025174404-572649812736 // indirect
 	lukechampine.com/blake3 v1.1.7 // indirect
 )

package/go.sum

@@ -107,8 +107,8 @@ github.com/yuin/goldmark v1.2.1/go.mod h1:3hX8gzYuyVAZsxl0MRgGTJEmQBFcNTphYh9dec
 golang.org/x/crypto v0.0.0-20190308221718-c2843e01d9a2/go.mod h1:djNgcEr1/C05ACkg1iLfiJU5Ep61QUkGW8qpdssI0+w=
 golang.org/x/crypto v0.0.0-20191011191535-87dc89f01550/go.mod h1:yigFU9vqHzYiE8UmvKecakEJjdnWj3jj499lnFckfCI=
 golang.org/x/crypto v0.0.0-20200622213623-75b288015ac9/go.mod h1:LzIPMQfyMNhhGPhUkYOs5KpL4U8rLKemX1yGLhDgUto=
-golang.org/x/crypto v0.0.0-20220525230936-793ad666bf5e h1:T8NU3HyQ8ClP4SEE+KbFlg6n0NhuTsN4MyznaarGsZM=
-golang.org/x/crypto v0.0.0-20220525230936-793ad666bf5e/go.mod h1:IxCIyHEi3zRg3s0A5j5BB6A9Jmi73HwBIUl50j+osU4=
+golang.org/x/crypto v0.1.1-0.20221024173537-a3485e174077 h1:t5bjOfJPQfaG9NV1imLZM5E2uzaLGs5/NtyMtRNVjQ4=
+golang.org/x/crypto v0.1.1-0.20221024173537-a3485e174077/go.mod h1:RecgLatLF4+eUMCP1PoPZQb+cVrJcOPbHkTkbkB9sbw=
 golang.org/x/mod v0.2.0/go.mod h1:s0Qsj1ACt9ePp/hMypM3fl4fZqREWJwdYDEqhRiZZUA=
 golang.org/x/mod v0.3.0/go.mod h1:s0Qsj1ACt9ePp/hMypM3fl4fZqREWJwdYDEqhRiZZUA=
 golang.org/x/net v0.0.0-20190404232315-eb5bcb51f2a3/go.mod h1:t9HGtf8HONx5eT2rtn7q6eTqICYqUVnKs3thJo3Qplg=
@@ -124,8 +124,8 @@ golang.org/x/sys v0.0.0-20200116001909-b77594299b42/go.mod h1:h1NjWce9XRLGQEsW7w
 golang.org/x/sys v0.0.0-20200930185726-fdedc70b468f/go.mod h1:h1NjWce9XRLGQEsW7wpKNCjG9DtNlClVuFLEZdDNbEs=
 golang.org/x/sys v0.0.0-20220704084225-05e143d24a9e/go.mod h1:oPkhp1MJrh7nUepCBck5+mAzfO9JrbApNNgaTdGDITg=
 golang.org/x/sys v0.0.0-20220715151400-c0bba94af5f8/go.mod h1:oPkhp1MJrh7nUepCBck5+mAzfO9JrbApNNgaTdGDITg=
-golang.org/x/sys v0.0.0-20220811171246-fbc7d0a398ab h1:2QkjZIsXupsJbJIdSjjUOgWK3aEtzyuh2mPt3l/CkeU=
-golang.org/x/sys v0.0.0-20220811171246-fbc7d0a398ab/go.mod h1:oPkhp1MJrh7nUepCBck5+mAzfO9JrbApNNgaTdGDITg=
+golang.org/x/sys v0.1.1-0.20221025174404-572649812736 h1:XFwPM8GRJFIl06GJimZwfwYKvzESqOQA2Jn17wVv7w8=
+golang.org/x/sys v0.1.1-0.20221025174404-572649812736/go.mod h1:oPkhp1MJrh7nUepCBck5+mAzfO9JrbApNNgaTdGDITg=
 golang.org/x/text v0.3.0/go.mod h1:NqM8EUOU14njkJ3fqMW+pc6Ldnwhi/IjpwHt7yyuwOQ=
 golang.org/x/text v0.3.2/go.mod h1:bEr9sfX3Q8Zfm5fL9x+3itogRgK3+ptLWKqgva+5dAk=
 golang.org/x/text v0.3.3/go.mod h1:5Zoc/QRtKVWzQhOtBMvqHzDpF6irO9z98xDceosuGiQ=

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "starpc",
-  "version": "0.13.0",
+  "version": "0.13.2",
   "description": "Streaming protobuf RPC service protocol over any two-way channel.",
   "license": "MIT",
   "author": {
@@ -59,12 +59,12 @@
     "singleQuote": true
   },
   "devDependencies": {
-    "@typescript-eslint/eslint-plugin": "^5.41.0",
-    "@typescript-eslint/parser": "^5.41.0",
+    "@typescript-eslint/eslint-plugin": "^5.42.0",
+    "@typescript-eslint/parser": "^5.42.0",
     "bufferutil": "^4.0.7",
     "depcheck": "^1.4.3",
-    "esbuild": "^0.15.12",
-    "eslint": "^8.26.0",
+    "esbuild": "^0.15.13",
+    "eslint": "^8.27.0",
     "eslint-config-prettier": "^8.5.0",
     "prettier": "^2.7.1",
     "rimraf": "^3.0.2",
@@ -73,9 +73,9 @@
     "utf-8-validate": "^5.0.10"
   },
   "dependencies": {
-    "@chainsafe/libp2p-yamux": "^3.0.2",
-    "@libp2p/interface-connection": "^3.0.2",
-    "@libp2p/interface-stream-muxer": "^3.0.0",
+    "@chainsafe/libp2p-yamux": "^3.0.3",
+    "@libp2p/interface-connection": "^3.0.3",
+    "@libp2p/interface-stream-muxer": "^3.0.1",
     "event-iterator": "^2.0.0",
     "is-promise": "^4.0.0",
     "isomorphic-ws": "^5.0.0",
@@ -85,10 +85,10 @@
     "it-pushable": "^3.1.0",
     "it-stream-types": "^1.0.4",
     "it-ws": "^5.0.3",
-    "long": "^5.2.0",
+    "long": "^5.2.1",
     "patch-package": "^6.5.0",
     "protobufjs": "^7.1.2",
     "uint8arraylist": "^2.3.3",
-    "ws": "^8.10.0"
+    "ws": "^8.11.0"
   }
 }

package/srpc/accept.go

@@ -3,19 +3,22 @@ package srpc
 import (
 	"context"
 	"net"
+
+	"github.com/libp2p/go-yamux/v4"
 )
 
 // AcceptMuxedListener accepts incoming connections from a net.Listener.
 //
 // Uses the default yamux muxer.
-func AcceptMuxedListener(ctx context.Context, lis net.Listener, srv *Server) error {
+// If yamux conf is nil, uses the defaults.
+func AcceptMuxedListener(ctx context.Context, lis net.Listener, srv *Server, yamuxConf *yamux.Config) error {
 	for {
 		nc, err := lis.Accept()
 		if err != nil {
 			return err
 		}
 
-		mc, err := NewMuxedConn(nc, false)
+		mc, err := NewMuxedConn(nc, false, yamuxConf)
 		if err != nil {
 			_ = nc.Close()
 			continue

package/srpc/muxed-conn.go

@@ -14,12 +14,18 @@ import (
 func NewYamuxConfig() *yamux.Config {
 	// Configuration options from go-libp2p-yamux:
 	config := *ymuxer.DefaultTransport.Config()
+	config.AcceptBacklog = 512
+	config.MaxMessageSize = 2e6
 	return &config
 }
 
 // NewMuxedConn constructs a new MuxedConn from a net.Conn.
-func NewMuxedConn(conn net.Conn, outbound bool) (network.MuxedConn, error) {
-	yamuxConf := NewYamuxConfig()
+//
+// If yamuxConf is nil, uses defaults.
+func NewMuxedConn(conn net.Conn, outbound bool, yamuxConf *yamux.Config) (network.MuxedConn, error) {
+	if yamuxConf == nil {
+		yamuxConf = NewYamuxConfig()
+	}
 
 	var sess *yamux.Session
 	var err error
@@ -36,13 +42,22 @@ func NewMuxedConn(conn net.Conn, outbound bool) (network.MuxedConn, error) {
 }
 
 // NewMuxedConnWithRwc builds a new MuxedConn with a io.ReadWriteCloser.
-func NewMuxedConnWithRwc(ctx context.Context, rwc io.ReadWriteCloser, outbound bool) (network.MuxedConn, error) {
-	return NewMuxedConn(NewRwcConn(ctx, rwc, nil, nil, 10), outbound)
+//
+// If yamuxConf is nil, uses defaults.
+func NewMuxedConnWithRwc(
+	ctx context.Context,
+	rwc io.ReadWriteCloser,
+	outbound bool,
+	yamuxConf *yamux.Config,
+) (network.MuxedConn, error) {
+	return NewMuxedConn(NewRwcConn(ctx, rwc, nil, nil, 10), outbound, yamuxConf)
 }
 
 // NewClientWithConn constructs the muxer and the client.
-func NewClientWithConn(conn net.Conn, outbound bool) (Client, error) {
-	mconn, err := NewMuxedConn(conn, outbound)
+//
+// if yamuxConf is nil, uses defaults.
+func NewClientWithConn(conn net.Conn, outbound bool, yamuxConf *yamux.Config) (Client, error) {
+	mconn, err := NewMuxedConn(conn, outbound, yamuxConf)
 	if err != nil {
 		return nil, err
 	}

package/srpc/server-http.go

@@ -32,16 +32,15 @@ func (s *HTTPServer) ServeHTTP(w http.ResponseWriter, r *http.Request) {
 
 	c, err := websocket.Accept(w, r, &websocket.AcceptOptions{})
 	if err != nil {
-		// TODO: handle / log error?
-		_ = err
+		w.WriteHeader(500)
+		w.Write([]byte(err.Error() + "\n"))
 		return
 	}
 	defer c.Close(websocket.StatusInternalError, "closed")
 
 	ctx := r.Context()
-	wsConn, err := NewWebSocketConn(ctx, c, true)
+	wsConn, err := NewWebSocketConn(ctx, c, true, nil)
 	if err != nil {
-		// TODO: handle / log error?
 		c.Close(websocket.StatusInternalError, err.Error())
 		return
 	}

package/srpc/websocket.go

@@ -5,6 +5,7 @@ import (
 	"io"
 
 	"github.com/libp2p/go-libp2p/core/network"
+	"github.com/libp2p/go-yamux/v4"
 	"nhooyr.io/websocket"
 )
 
@@ -17,9 +18,11 @@ type WebSocketConn struct {
 }
 
 // NewWebSocketConn constructs a new WebSocket connection.
-func NewWebSocketConn(ctx context.Context, conn *websocket.Conn, isServer bool) (*WebSocketConn, error) {
+//
+// if yamuxConf is unset, uses the defaults.
+func NewWebSocketConn(ctx context.Context, conn *websocket.Conn, isServer bool, yamuxConf *yamux.Config) (*WebSocketConn, error) {
 	nc := websocket.NetConn(ctx, conn, websocket.MessageBinary)
-	muxedConn, err := NewMuxedConn(nc, !isServer)
+	muxedConn, err := NewMuxedConn(nc, !isServer, yamuxConf)
 	if err != nil {
 		return nil, err
 	}