review: leave old apidump file permissions

pawbana · pawbana · commit e7bd5a3f5278 · 2026-04-13T11:40:42.000Z
diff --git a/intercept/apidump/apidump.go b/intercept/apidump/apidump.go
@@ -110,7 +110,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 {
@@ -133,7 +133,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
`@@ -110,7 +110,7 @@ func (d dumper) dumpRequest(req http.Request) error {`
`110`	`110`	`buf.Write(prettyBody)`
`111`	`111`	`buf.WriteByte('\n')`
`112`	`112`
`113`		`- return os.WriteFile(dumpPath, buf.Bytes(), 0o600)`
	`113`	`+ return os.WriteFile(dumpPath, buf.Bytes(), 0o644) //nolint:gosec // https://github.com/coder/aibridge/pull/256#discussion_r3072143983`
`114`	`114`	`}`
`115`	`115`
`116`	`116`	`func (d dumper) dumpResponse(resp http.Response) error {`
`@@ -133,7 +133,7 @@ func (d dumper) dumpResponse(resp http.Response) error {`
`133`	`133`
`134`	`134`	`if resp.Body == nil {`
`135`	`135`	`// No body, just write headers`
`136`		`- return os.WriteFile(dumpPath, headerBuf.Bytes(), 0o600)`
	`136`	`+ return os.WriteFile(dumpPath, headerBuf.Bytes(), 0o644) //nolint:gosec // https://github.com/coder/aibridge/pull/256#discussion_r3072143983`
`137`	`137`	`}`
`138`	`138`
`139`	`139`	`// Wrap the response body to capture it as it streams`