vault icon indicating copy to clipboard operation
vault copied to clipboard

fix an edge case bug that "identity_policies" is nil

Open tianhaopx opened this issue 2 years ago • 2 comments

Fixes #17006

client side: check if "identity_policies" is nil, goto DONE, then cli vault login will not return unexpected error.

server side: check if key out.NamespaceID exists in the map identityPolicies returned from func fetchEntityAndDerivedPolicies

tianhaopx avatar Sep 03 '22 10:09 tianhaopx

CLA assistant check
All committers have signed the CLA.

hashicorp-cla avatar Sep 03 '22 10:09 hashicorp-cla

Hi @tianhaopx, thanks for the submission. It looks good, I just wonder if you might be able to add a test to secret_test.go to check the return from TokenPolicies? I think we're good to merge it in after that. :)

peteski22 avatar Sep 08 '22 18:09 peteski22