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.

Sign up for GitHub

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

Jump to bottom

elasticsearch-rest-client host options only work if autowiring is disabled #6163

Closed
jamesnetherton opened this issue Jun 6, 2024 · 1 comment
Closed

elasticsearch-rest-client host options only work if autowiring is disabled #6163

jamesnetherton opened this issue Jun 6, 2024 · 1 comment
Assignees
@jamesnetherton
Labels
area/elasticsearch-rest-client bug Something isn't working
Milestone
3.12.0

Comments

@jamesnetherton
Copy link
Contributor

Bug description

If endpoint options like hostAddressesList, user & password are set, they get ignored because the the Quarkus generated RestClient bean gets autowired. Unless quarkus.elasticsearch config is provided, the client will default to connecting on localhost:9200 with no authentication.
@jamesnetherton jamesnetherton added the bug Something isn't working label Jun 6, 2024
@jamesnetherton jamesnetherton self-assigned this Jun 6, 2024
@jamesnetherton
Copy link
Contributor Author

Unless we want to add native support for this extension, the simplest solution is to remove the quarkus-elasticsearch-rest-client dependency as it's not doing anything useful for us.

Else, it should be enough to only depend on quarkus-elasticsearch-rest-client-common, to avoid pulling in the entirety of Quarkus ES functionality.

@orpiske orpiske mentioned this issue Jun 6, 2024
Closed
jamesnetherton added a commit to jamesnetherton/camel-quarkus that referenced this issue Jun 7, 2024
@jamesnetherton
Increase elasticsearch-rest-client test coverage
9909d3b 
Fixes apache#5737
Fixes apache#6163
Fixes apache#6164
@github-actions github-actions bot added this to the 3.12.0 milestone Jun 10, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@jamesnetherton jamesnetherton

Labels
area/elasticsearch-rest-client bug Something isn't working
Projects
None yet
Milestone
3.12.0
Development

No branches or pull requests
1 participant
@jamesnetherton