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

[Fleet] Using a * as the first character in a password throws a YAML validation error in the policy editor #176596

Closed
kpollich opened this issue Feb 9, 2024 · 2 comments
Labels
bug Fixes for quality problems that affect the customer experience Team:Fleet Team label for Observability Data Collection Fleet team

Comments

@kpollich
Copy link
Member

kpollich commented Feb 9, 2024

To reproduce

  1. Create a package policy for the elasticsearch integration
  2. Add a password beginning with * to the auth section under advanced options
  3. Observe the validation error in the toast that appears

image

This looks like an issue with policy compilation or YAML validation in some capacity. We've seen similar issues with special character in passwords before, which I believe we've solved by wrapping fields in single quotes, e.g. #127268

@kpollich kpollich added bug Fixes for quality problems that affect the customer experience Team:Fleet Team label for Observability Data Collection Fleet team labels Feb 9, 2024
@elasticmachine
Copy link
Contributor

Pinging @elastic/fleet (Team:Fleet)

@kpollich
Copy link
Member Author

kpollich commented Feb 9, 2024

The Elasticsearch integration needs to wrap its password fields in escape_string

https://github.com/elastic/integrations/blob/main/packages/elasticsearch/data_stream/node/agent/stream/stream.yml.hbs#L10-L15

Closing as this can be fixed on the integration side.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Fixes for quality problems that affect the customer experience Team:Fleet Team label for Observability Data Collection Fleet team
Projects
None yet
Development

No branches or pull requests

2 participants