-
Notifications
You must be signed in to change notification settings - Fork 8.3k
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
[Obs AI Assistant] Manual migration from access tags to authz #202486
[Obs AI Assistant] Manual migration from access tags to authz #202486
Conversation
Pinging @elastic/obs-ai-assistant (Team:Obs AI Assistant) |
b88f800
to
a5a5a25
Compare
🤖 GitHub commentsExpand to view the GitHub comments
Just comment with:
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm. Do we need a generic auth test that verifies the privilege of routes?
Thanks @sorenlouv |
As discussed offline, yes that would be nice. Just something super simple. |
…-ai-assistant routes
a5a5a25
to
587dcb7
Compare
💚 Build Succeeded
Metrics [docs]
History
cc @viduni94 |
The latest changes I pushed are not being reflected in this PR, closing this and re-opening a new PR |
New PR - #202817 |
Summary
Problem
tags: [access:ai_assistant]
is deprecated.Solution
All the routes that use this tag needs to be migrated to the
authz
:requiredPrivileges
property.Checklist
release_note:*
label is applied per the guidelines