You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
There is an issue in the common-utils repo (opensearch-project/common-utils#97) to move this much-copied file from multiple plugin repos into common-utils. I am opening an issue in the core repo to discuss adding the common methods directly into the core repo in a generic way that is not tied with the security plugin. Many of the methods that are copied between plugins have to do with setting up a REST Client to make rest requests against a cluster with the security plugin installed and only accepting requests using HTTPS.
When OpenSearch uses HTTPS, REST Clients must be configured with a truststore containing the root certificate or be configured to not validate SSL (equivalent of curl's --insecure). From looking at a few plugin repos, it looks like the latter overly trustworthy approach is the most commonly used.
Potential methods that could be abstracted into core (taking methods from k-NN as an example):
There is an issue in the common-utils repo (opensearch-project/common-utils#97) to move this much-copied file from multiple plugin repos into common-utils. I am opening an issue in the core repo to discuss adding the common methods directly into the core repo in a generic way that is not tied with the security plugin. Many of the methods that are copied between plugins have to do with setting up a REST Client to make rest requests against a cluster with the security plugin installed and only accepting requests using HTTPS.
When OpenSearch uses HTTPS, REST Clients must be configured with a truststore containing the root certificate or be configured to not validate SSL (equivalent of curl's
--insecure
). From looking at a few plugin repos, it looks like the latter overly trustworthy approach is the most commonly used.Potential methods that could be abstracted into core (taking methods from k-NN as an example):
The text was updated successfully, but these errors were encountered: