Merge pull request #18 from sas1024/master

Add JSON-RPC2 notification support for generated go client
vmkteam · Oct 4, 2022 · c9fcb8e · c9fcb8e
2 parents 3de8af1 + 10a2384
commit c9fcb8e
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 0 deletions.
diff --git a/golang/go_template.go b/golang/go_template.go
@@ -159,6 +159,10 @@ func (c *rpcClient) call(ctx context.Context, methodName string, request, result
 
 // Exec makes http request to jsonrpc endpoint and returns json rpc response.
 func (rc *rpcClient) Exec(ctx context.Context, rpcReq zenrpc.Request) (*zenrpc.Response, error) {
+	if n, ok := ctx.Value("JSONRPC2-Notification").(bool); ok && n {
+		rpcReq.ID = nil
+	}
+
 	c, err := json.Marshal(rpcReq)
 	if err != nil {
 		return nil, fmt.Errorf("json marshal call failed: %w", err)

diff --git a/golang/testdata/catalogue_client.go b/golang/testdata/catalogue_client.go