diff --git a/changelog/18568.txt b/changelog/18568.txt new file mode 100644 index 000000000000..a1fbabf2545a --- /dev/null +++ b/changelog/18568.txt @@ -0,0 +1,3 @@ +```release-note:bug +core: Fix spurious `permission denied` for all HelpOperations on sudo-protected paths +``` diff --git a/vault/acl.go b/vault/acl.go index 7aeb102bdab8..3a3f99a4c6ae 100644 --- a/vault/acl.go +++ b/vault/acl.go @@ -719,7 +719,9 @@ func (c *Core) performPolicyChecks(ctx context.Context, acl *ACL, te *logical.To if !ret.ACLResults.Allowed { return ret } - if !ret.RootPrivs && opts.RootPrivsRequired { + // Since HelpOperation was fast-pathed inside AllowOperation, RootPrivs will not have been populated in this + // case, so we need to special-case that here as well, or we'll block HelpOperation on all sudo-protected paths. + if !ret.RootPrivs && opts.RootPrivsRequired && req.Operation != logical.HelpOperation { return ret } }