Skip to content

Commit bb9b258

Browse files
authored
fix(bundler): Only close multipart writer once (#1528)
1 parent e7aa4bd commit bb9b258

File tree

1 file changed

+3
-5
lines changed

1 file changed

+3
-5
lines changed

internal/bundler/upload.go

Lines changed: 3 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -41,22 +41,20 @@ func (up *Uploader) Validate() error {
4141

4242
func (up *Uploader) buildRequest(ctx context.Context, result map[string]string) (*http.Request, error) {
4343
body := bytes.NewBuffer([]byte{})
44-
4544
w := multipart.NewWriter(body)
46-
defer w.Close()
4745
if err := writeInputs(w, up.configPath, up.config); err != nil {
4846
return nil, err
4947
}
5048
if err := writeOutputs(w, up.dir, result); err != nil {
5149
return nil, err
5250
}
53-
w.Close()
54-
51+
if err := w.Close(); err != nil {
52+
return nil, err
53+
}
5554
req, err := http.NewRequest("POST", "https://api.sqlc.dev/upload", body)
5655
if err != nil {
5756
return nil, err
5857
}
59-
6058
// Set sqlc-version header
6159
req.Header.Set("Content-Type", w.FormDataContentType())
6260
req.Header.Set("Authorization", fmt.Sprintf("Bearer %s", up.token))

0 commit comments

Comments
 (0)