fix(coderd): prevent nil err deref (coder#12475)

johnstcn · web-flow · commit 0647ec1960ae · 2024-03-11T14:03:58.000Z
diff --git a/coderd/files.go b/coderd/files.go
@@ -175,7 +175,6 @@ func (api *API) fileByID(rw http.ResponseWriter, r *http.Request) {
 		if file.Mimetype != codersdk.ContentTypeTar {
 			httpapi.Write(ctx, rw, http.StatusBadRequest, codersdk.Response{
 				Message: "Only .tar files can be converted to .zip format",
-				Detail:  err.Error(),
 			})
 			return
 		}
@@ -192,7 +191,6 @@ func (api *API) fileByID(rw http.ResponseWriter, r *http.Request) {
 	default:
 		httpapi.Write(ctx, rw, http.StatusBadRequest, codersdk.Response{
 			Message: "Unsupported conversion format.",
-			Detail:  err.Error(),
 		})
 	}
 }

Original file line number	Diff line number	Diff line change
`@@ -175,7 +175,6 @@ func (api API) fileByID(rw http.ResponseWriter, r http.Request) {`
`175`	`175`	`if file.Mimetype != codersdk.ContentTypeTar {`
`176`	`176`	`httpapi.Write(ctx, rw, http.StatusBadRequest, codersdk.Response{`
`177`	`177`	`Message: "Only .tar files can be converted to .zip format",`
`178`		`- Detail: err.Error(),`
`179`	`178`	`})`
`180`	`179`	`return`
`181`	`180`	`}`
`@@ -192,7 +191,6 @@ func (api API) fileByID(rw http.ResponseWriter, r http.Request) {`
`192`	`191`	`default:`
`193`	`192`	`httpapi.Write(ctx, rw, http.StatusBadRequest, codersdk.Response{`
`194`	`193`	`Message: "Unsupported conversion format.",`
`195`		`- Detail: err.Error(),`
`196`	`194`	`})`
`197`	`195`	`}`
`198`	`196`	`}`