laantungir/nak
1
0
Fork 0
mirror of https://github.com/fiatjaf/nak.git synced 2026-07-22 07:48:27 +00:00
Code Issues Packages Projects Releases Wiki Activity

admin: close the response body on error paths too, and don't use the response body as a format string.

Browse Source
This commit is contained in:
Yasuhiro Matsumoto
2026-07-15 08:35:58 +09:00
parent 1cb730f34d
commit c525a25e51
1 changed files with 3 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files

6
admin.go
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")