Skip to content

Commit

Permalink
ci: address return value is not checked (errcheck)
Browse files Browse the repository at this point in the history
Signed-off-by: Praveen M <[email protected]>
  • Loading branch information
iPraveenParihar committed Dec 20, 2024
1 parent 719fef9 commit 1c39ace
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 2 deletions.
2 changes: 1 addition & 1 deletion internal/kms/vault.go
Original file line number Diff line number Diff line change
Expand Up @@ -305,7 +305,7 @@ func (vc *vaultConnection) Destroy() {
tmpFile, ok := vc.vaultConfig[api.EnvVaultCACert]
if ok {
// ignore error on failure to remove tmpfile (gosec complains)
//nolint:forcetypeassert // ignore error on failure to remove tmpfile
//nolint:forcetypeassert,errcheck // ignore error on failure to remove tmpfile
_ = os.Remove(tmpFile.(string))
}
}
Expand Down
2 changes: 1 addition & 1 deletion internal/kms/vault_tokens_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -119,7 +119,7 @@ func TestInitVaultTokensKMS(t *testing.T) {
// add tenant "bob"
bob := make(map[string]interface{})
bob["vaultAddress"] = "https://vault.bob.example.org"
//nolint:forcetypeassert // as its a test we dont need to check assertion here.
//nolint:forcetypeassert,errcheck // as its a test we dont need to check assertion here.
args.Config["tenants"].(map[string]interface{})["bob"] = bob

_, err = initVaultTokensKMS(args)
Expand Down

0 comments on commit 1c39ace

Please sign in to comment.