upstream/vault
1
0
Fork 0
mirror of https://github.com/hashicorp/vault.git synced 2026-02-03 20:40:45 -05:00
Code Issues Projects Releases Packages Wiki Activity Actions 11

add nil check for mfa enforcement config namespace on login (#20375)

Browse source 
* add nil check for mfa enforcement config ns

* move nil check and add changelog
This commit is contained in:
davidadeleon 2023-04-26 16:52:39 -04:00 committed by GitHub
parent d766848664
commit c261445f83
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23
2 changed files with 5 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

3
changelog/20375.txt Normal file
View file

@ -0,0 +1,3 @@
```release-note:bug
core: prevent panic on login after namespace is deleted that had mfa enforcement
```

3
vault/login_mfa.go
View file

@ -1788,7 +1788,8 @@ ECONFIG_LOOP:
if err != nil {
return nil, fmt.Errorf("failed to find the MFAEnforcementConfig namespace")
}
if eConfig == nil || (eConfigNS.ID != ns.ID && !ns.HasParent(eConfigNS)) {
if eConfig == nil || eConfigNS == nil || (eConfigNS.ID != ns.ID && !ns.HasParent(eConfigNS)) {
continue
}