Skip to content

Commit 260a210

Browse files
authored
Merge pull request #502 from k1LoW/fix/gosec-lint-errors
fix: resolve gosec lint errors in client.go
2 parents 3f1f920 + 24b82d4 commit 260a210

File tree

1 file changed

+3
-1
lines changed

1 file changed

+3
-1
lines changed

client.go

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -180,7 +180,9 @@ func (d *Deck) getTokenFromWeb(ctx context.Context, config *oauth2.Config) (_ *o
180180
}
181181
state := base64.RawURLEncoding.EncodeToString(stateBytes)
182182
listenCtx, listening := context.WithCancel(ctx)
183+
defer listening()
183184
doneCtx, done := context.WithCancel(ctx)
185+
defer done()
184186
// run and stop local server
185187
handler := http.NewServeMux()
186188
handler.HandleFunc("/", func(w http.ResponseWriter, r *http.Request) {
@@ -274,7 +276,7 @@ func (d *Deck) saveToken(path string, token *oauth2.Token) (err error) {
274276
return fmt.Errorf("unable to cache oauth token: %w", err)
275277
}
276278
defer f.Close()
277-
if err := json.NewEncoder(f).Encode(token); err != nil {
279+
if err := json.NewEncoder(f).Encode(token); err != nil { //nolint:gosec
278280
return fmt.Errorf("unable to cache oauth token: %w", err)
279281
}
280282
return nil

0 commit comments

Comments
 (0)