Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Go through permission checks and rework to generated detailed information. #12240

Closed
markan opened this issue Feb 1, 2022 · 0 comments · Fixed by #12470
Closed

Go through permission checks and rework to generated detailed information. #12240

markan opened this issue Feb 1, 2022 · 0 comments · Fixed by #12470
Assignees
Labels
theme/acls ACL and token generation

Comments

@markan
Copy link
Contributor

markan commented Feb 1, 2022

This builds on #12239

We currently don't give very detailed information when acl checks fail. We should fix this by going through our code and wherever possible capturing the full information available at the point of error construction. This intended to be relatively low impact work and should not include cases where the enforcement decision is not near the construction of the error.

Note: For review purposes this work will probably be broken down into smaller PRs on a subsystem basis.

@markan markan self-assigned this Feb 1, 2022
@markan markan added theme/acls ACL and token generation theme/internal-cleanup Used to identify tech debt, testing improvements, code refactoring, and non-impactful optimization and removed theme/internal-cleanup Used to identify tech debt, testing improvements, code refactoring, and non-impactful optimization labels Feb 1, 2022
@markan markan linked a pull request Mar 1, 2022 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
theme/acls ACL and token generation
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant