Merge pull request #184 from mattn/fix/admin-body-leak

admin: close the response body and don't use it as a format string
This commit is contained in:
mattn
2026-07-15 09:27:10 +00:00
committed by GitHub
+3 -3
View File
@@ -132,6 +132,7 @@ var admin = &cli.Command{
continue
}
b, err := io.ReadAll(resp.Body)
resp.Body.Close()
if err != nil {
log("failed to read response: %s\n", err)
continue
@@ -142,7 +143,7 @@ var admin = &cli.Command{
if len(bodyPrintable) > 300 {
bodyPrintable = bodyPrintable[0:297] + "..."
}
log(bodyPrintable)
log("%s", bodyPrintable)
continue
}
var response nip86.Response
@@ -152,10 +153,9 @@ var admin = &cli.Command{
if len(bodyPrintable) > 300 {
bodyPrintable = bodyPrintable[0:297] + "..."
}
log(bodyPrintable)
log("%s", bodyPrintable)
continue
}
resp.Body.Close()
// print the result
log("\n")