-
Notifications
You must be signed in to change notification settings - Fork 429
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
[Asset Inventory] Organization Support #11398
base: main
Are you sure you want to change the base?
Conversation
- name: azure.credentials.client_password | ||
type: password | ||
title: Client Password | ||
multi: false | ||
required: false | ||
show_user: true | ||
show_user: false |
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.
Why show_user for client username and client password became false?
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.
IMHO they are not used for any of the authentication kinds. Let me know if they are indeed used somewhere and I missed it.
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.
Quality Gate failedFailed conditions |
💚 Build Succeeded
History
cc @kubasobon |
Proposed commit message
The PR:
Checklist
changelog.yml
file.Related issues
Fixes https://github.com/elastic/security-team/issues/10606
Screenshots
Screenshot showing the new deployment type drop-down and credential field descriptions