-
Notifications
You must be signed in to change notification settings - Fork 433
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
Add the possibility to use secure es with basic auth #187
Conversation
@confluentinc It looks like @elarib just signed our Contributor License Agreement. 👍 Always at your service, clabot |
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
HttpHost host = new HttpHost(url.getHost(), url.getPort(), url.getProtocol()); | ||
hostSet.add(host); | ||
} catch (MalformedURLException e) { | ||
e.printStackTrace(); |
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.
Print exception together with an error message rather than just dumping it?
log.warn("Failed to parse malformed url", e);
Would be nice to add support for basic auth to elasticsearch by finishing this PR! (or one of the others) Can we expect it anytime soon? |
Sorry, it seems like we've had a number of duplicates. Basic auth has already been merged via #216, so I'm closing this issue. |
There is some use cases of secure ES (for example multitenancy), so the Jest client should authenticate.
This PR add the possibility to sink data to a secure ES with basic auth.
✅Code changed & tested
✅Doc changed
FYI : There are many tools used to secure Elasticsearch, for example
xPack https://www.elastic.co/guide/en/x-pack/current/security-getting-started.html
Searchguard https://docs.search-guard.com/latest/index