@wonderful-quill-11255 We’re in a situation were we’d need to add some BasicAuth over HTTPS functionality to RestHighLevelClientFactory.java. I think the idea would be to conditionally add a BasicCredenetialsProvider (like here) . Do you see any problems with this?

@high-hospital-85984 Sounds good to me!

Thanks! I’ll ping you in the PR to make sure I dont break the SSL functionality (I dont have a good way of testing it currently)

Nice. Regarding the testing I had the same issue so I just ran the quickstart stack but repointed the ES using components to our real non-prod ES. Might be worth it to have an ES docker config that starts up in SSL+auth mode but I was reluctant to add that before.

Ok, I have a suggestion here: https://github.com/linkedin/datahub/pull/2303 It would be great if you could review and test it against your ES instance.

Nice! Sorry, didn’t see it until now, but I will test asap

Username/password works at least (tested). I hope I didn’t break the private key path, though.

I can confirm that username/password works. Although the password cannot contain certain characters, e.g

#

or

@

. I haven’t tested out the more advanced SSL options like keystore etc.

Nice catch!

Great job and thanks a lot for this feature btw!