opensearch-project / opensearch-js

Node.js Client for OpenSearch
https://opensearch.org/docs/latest/clients/javascript/
Apache License 2.0
179 stars 118 forks source link

[v2.12.0] Ensure CI/documentation reflect changes to default admin credentials #699

Closed DarshitChanpura closed 4 months ago

DarshitChanpura commented 5 months ago

Background

Previously, when installing the security plugin demo configuration, the cluster was spun up with the default admin credentials, admin:admin. A change was made in main and backported to 2.x for the 2.12.0 release, which now requires an initial admin password to be passed in via the environment variable OPENSEARCH_INITIAL_ADMIN_PASSWORD. This will break some CI/testing that relies on OpenSearch to come up without setting this environment variable. This tracking issue is to ensure compliance with the new changes.

Coming from: https://github.com/opensearch-project/security/issues/3624

Acceptance Criteria

wbeckler commented 5 months ago

It looks like there are at least nine places where we need to fix things: https://github.com/search?q=repo%3Aopensearch-project%2Fopensearch-js+username+admin&type=code

wbeckler commented 5 months ago

https://github.com/opensearch-project/opensearch-js/pull/669