fix(ldap/auth): handle nil Auth struct when authentication fails (#428)

go-vela · Nov 7, 2023 · 669ea15 · 669ea15
1 parent cc2ce1d
commit 669ea15
Showing 1 changed file with 5 additions and 0 deletions.
diff --git a/vault/vault.go b/vault/vault.go
@@ -84,6 +84,11 @@ func New(s *Setup) (*Client, error) {
 			return nil, fmt.Errorf("unable to get user token: %w", err)
 		}
 
+		// vault will return a nil Auth struct with no error if path is correct but password fails
+		if user.Auth == nil {
+			return nil, fmt.Errorf("unable to set user token: authentication failed")
+		}
+
 		// set Vault API token in client
 		vault.SetToken(user.Auth.ClientToken)