Improve Elasticsearch ServiceConnection #35926
Currently only Elasticsearch 7.x works with @ServiceConnection without any additional (manual) configuration or disabling security and SSL.
The proposed changes make it possible to use also: * version 8.x (where security and SSL are enabled by default) * version 7.x with with security and SSL enabled.
It is achieved by: * detecting if SSL is in place (therefore needing HTTPS) * security is enabled (thus requiring username and password)
Comment From: pivotal-cla
@pioorg Please sign the Contributor License Agreement!
Click here to manually synchronize the status of this Pull Request.
See the FAQ for frequently asked questions.
Comment From: wilkinsona
Thanks, @pioorg. Unfortunately, as indicated by it being assigned to someone, #35926 is already being worked on. There's also an existing prototype linked to in the issue's comments.
Comment From: pioorg
Hello @wilkinsona I saw both prototypes mentioned in the issue and the issue is that (in my understanding), they won't work, because they assume security and HTTPS always come together, and are always present/mandatory in 8.x, and never working in 7.x. This assumption is not correct, I'm afraid. I also made a comment in the issue. I'm not interested in this PR being merged. What I'm after is having a working and correct solution. Thank you!