review: leave old apidump file permissions

pawbana · pawbana · commit 3a516ee2c2ff · 2026-04-13T15:22:29.000Z
diff --git a/intercept/apidump/apidump.go b/intercept/apidump/apidump.go
@@ -109,7 +109,7 @@ func (d *dumper) dumpRequest(req *http.Request) error {
 	buf.Write(prettyBody)
 	buf.WriteByte('\n')
 
-	return os.WriteFile(dumpPath, buf.Bytes(), 0o600)
+	return os.WriteFile(dumpPath, buf.Bytes(), 0o644) //nolint:gosec // https://github.com/coder/aibridge/pull/256#discussion_r3072143983
 }
 
 func (d *dumper) dumpResponse(resp *http.Response) error {
@@ -132,7 +132,7 @@ func (d *dumper) dumpResponse(resp *http.Response) error {
 
 	if resp.Body == nil {
 		// No body, just write headers
-		return os.WriteFile(dumpPath, headerBuf.Bytes(), 0o600)
+		return os.WriteFile(dumpPath, headerBuf.Bytes(), 0o644) //nolint:gosec // https://github.com/coder/aibridge/pull/256#discussion_r3072143983
 	}
 
 	// Wrap the response body to capture it as it streams

Original file line number	Diff line number	Diff line change
`@@ -109,7 +109,7 @@ func (d dumper) dumpRequest(req http.Request) error {`
`109`	`109`	`buf.Write(prettyBody)`
`110`	`110`	`buf.WriteByte('\n')`
`111`	`111`
`112`		`- return os.WriteFile(dumpPath, buf.Bytes(), 0o600)`
	`112`	`+ return os.WriteFile(dumpPath, buf.Bytes(), 0o644) //nolint:gosec // https://github.com/coder/aibridge/pull/256#discussion_r3072143983`
`113`	`113`	`}`
`114`	`114`
`115`	`115`	`func (d dumper) dumpResponse(resp http.Response) error {`
`@@ -132,7 +132,7 @@ func (d dumper) dumpResponse(resp http.Response) error {`
`132`	`132`
`133`	`133`	`if resp.Body == nil {`
`134`	`134`	`// No body, just write headers`
`135`		`- return os.WriteFile(dumpPath, headerBuf.Bytes(), 0o600)`
	`135`	`+ return os.WriteFile(dumpPath, headerBuf.Bytes(), 0o644) //nolint:gosec // https://github.com/coder/aibridge/pull/256#discussion_r3072143983`
`136`	`136`	`}`
`137`	`137`
`138`	`138`	`// Wrap the response body to capture it as it streams`