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

feat: rbac middleware #26347

Merged
merged 12 commits into from
Nov 27, 2024
Merged

feat: rbac middleware #26347

merged 12 commits into from
Nov 27, 2024

Conversation

zlwaterfield
Copy link
Contributor

@zlwaterfield zlwaterfield commented Nov 21, 2024

This reverts commit b49e024.

Re-reverting the RBAC PR with a few changes to resolve the issue that caused the revert.

This PR makes one extra change to solve the reason for the revert, see commit here.

More context in Slack

@zlwaterfield zlwaterfield self-assigned this Nov 21, 2024
@zlwaterfield zlwaterfield changed the title rbac middleware feat: rbac middleware Nov 21, 2024
@zlwaterfield zlwaterfield marked this pull request as ready for review November 25, 2024 23:52
@zlwaterfield zlwaterfield merged commit dd56af5 into master Nov 27, 2024
91 checks passed
@zlwaterfield zlwaterfield deleted the revert/revert/zach/rbac/3 branch November 27, 2024 16:26
Copy link

sentry-io bot commented Nov 27, 2024

Suspect Issues

This pull request was deployed and Sentry observed the following issues:

  • ‼️ OperationalError: consuming input failed: query_wait_timeout /api/feature_flag/local_evaluation/ View Issue
  • ‼️ ValidationError: '“%(value)s” is not a valid UUID.' /.* View Issue
  • ‼️ AttributeError: 'NoneType' object has no attribute 'id' /api/organizations/{parent_lookup_organization_... View Issue
  • ‼️ OperationalError: consuming input failed: query_wait_timeout /api/feature_flag/local_evaluation/ View Issue
  • ‼️ OperationalError: consuming input failed: query_wait_timeout /api/feature_flag/local_evaluation/ View Issue

Did you find this useful? React with a 👍 or 👎

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants