New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

metrics/librato: ensure resp.body closed #26969

Merged

holiman merged 16 commits into ethereum:master from jsvisa:metrics-librato-resp-close

Mar 27, 2023

node: ensure rpc.body.close

Signed-off-by: jsvisa <[email protected]>

Loading branch information

jsvisa committed Mar 24, 2023

commit ea1c869e081f9ae7101e7be9529c873f1aacdf06

node/rpcstack_test.go

-Original file line number
+Diff line change
@@ Expand Up @@
     	if err != nil {
     		t.Fatal(err)
     	}
+    	t.Cleanup(func() { resp.Body.Close() })
     	return resp
     }
@@ Expand Down @@