Skip to content

Commit

Permalink
SECS-8514/rm_JWT_from_atlantis_comment (#20)
Browse files Browse the repository at this point in the history
* Trimm logged jwt, display only 5 first char

* remove all reference to jwt in error messages
  • Loading branch information
jeanblablatiste authored Aug 28, 2024
1 parent 00ac529 commit 2a93c8f
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions internal/provider/provider.go
Original file line number Diff line number Diff line change
Expand Up @@ -151,11 +151,11 @@ func setupVaultClientAuth(client *vault.Client, authConf *providerAuthModel) err
path := authConf.Path.ValueString()
authInfo, err := client.Logical().Write(path, loginData)
if err != nil {
return fmt.Errorf("unable to log in with Vault Kubernetes authentication with role %s and JWT %s: %w", role, jwt, err)
return fmt.Errorf("unable to log in with Vault Kubernetes authentication with role %s and err : %w", role, err)
}

if authInfo == nil {
return fmt.Errorf("not auth info returned for kubernetes auth with role %s and JWT %s: %s", role, jwt, err)
return fmt.Errorf("not auth info returned for kubernetes auth with role %s and err : %s", role, err)
}

if authInfo.Auth == nil || authInfo.Auth.ClientToken == "" {
Expand Down

0 comments on commit 2a93c8f

Please sign in to comment.