Skip to content

fix: resolve gosec lint errors in client.go#502

Merged
k1LoW merged 1 commit intomainfrom
fix/gosec-lint-errors
Mar 10, 2026
Merged

fix: resolve gosec lint errors in client.go#502
k1LoW merged 1 commit intomainfrom
fix/gosec-lint-errors

Conversation

@k1LoW
Copy link
Owner

@k1LoW k1LoW commented Mar 10, 2026

Summary

  • Add defer for context.WithCancel cancel functions to fix G118 (cancel function not called) warnings
  • Add //nolint:gosec for intentional OAuth token file encoding to suppress G117 (secret pattern match) false positive

- Add defer for context cancel functions to fix G118 warnings
- Add nolint:gosec for intentional token file encoding to fix G117
@k1LoW k1LoW self-assigned this Mar 10, 2026
@github-actions
Copy link
Contributor

Code Metrics Report

main (15b675a) #502 (43a20ac) +/-
Coverage 33.8% 33.7% -0.1%
Code to Test Ratio 1:0.9 1:0.9 -0.1
Test Execution Time 11s 11s 0s
Details
  |                     | main (15b675a) | #502 (43a20ac) |  +/-  |
  |---------------------|----------------|----------------|-------|
- | Coverage            |          33.8% |          33.7% | -0.1% |
  |   Files             |             31 |             31 |     0 |
  |   Lines             |           3405 |           3407 |    +2 |
  |   Covered           |           1151 |           1151 |     0 |
- | Code to Test Ratio  |          1:0.9 |          1:0.9 |  -0.1 |
  |   Code              |           7020 |           7022 |    +2 |
  |   Test              |           6581 |           6581 |     0 |
  | Test Execution Time |            11s |            11s |    0s |

Code coverage of files in pull request scope (0.0% → 0.0%)

Files Coverage +/- Status
client.go 0.0% 0.0% modified

Reported by octocov

@k1LoW k1LoW merged commit 260a210 into main Mar 10, 2026
1 check passed
@k1LoW k1LoW deleted the fix/gosec-lint-errors branch March 10, 2026 13:27
@github-actions github-actions bot mentioned this pull request Mar 9, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant