Skip to content
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

[2.12.0] Ensure compatibility for 2.12.0 default admin credentials changes #65

Closed
derek-ho opened this issue Dec 28, 2023 · 3 comments
Closed
Labels
enhancement New feature or request untriaged

Comments

@derek-ho
Copy link
Collaborator

Is your feature request related to a problem?
With the 2.12.0 release of OpenSearch, the security plugin's install demo configuration script requires an initial admin password to be passed in to run. We want to ensure all repos are aware/complaint with this change.

What solution would you like?
A clear and concise description of what you want to happen.

What alternatives have you considered?
A clear and concise description of any alternative solutions or features you've considered.

Do you have any additional context?
Add any other context or screenshots about the feature request here.

@derek-ho derek-ho added enhancement New feature or request untriaged labels Dec 28, 2023
@derek-ho
Copy link
Collaborator Author

Seems like tests are not being run:

# Tests are disabled (commented out) in all jobs because they are fail and/or outdated
, if maintainers have no objections I will close this issue out.

@joshuali925
Copy link
Member

TEST_F(TestSQLDriverConnect, ConnStringAbbrevsUID) {
std::wstring abbrev_str =
use_ssl ? L"Driver={OpenSearch ODBC};"
L"host=https://localhost;port=9200;"
L"UID=admin;password=admin;auth=BASIC;useSSL=1;"
L"hostnameVerification=0;logLevel=0;logOutput=C:\\;"
L"responseTimeout=10;"
: L"Driver={OpenSearch ODBC};"
L"host=localhost;port=9200;"
L"UID=admin;password=admin;auth=BASIC;useSSL=0;"
L"hostnameVerification=0;logLevel=0;logOutput=C:\\;"
L"responseTimeout=10;";
SQLRETURN ret =
SQLDriverConnect(m_conn, NULL, (SQLTCHAR*)abbrev_str.c_str(), SQL_NTS,
m_out_conn_string, IT_SIZEOF(m_out_conn_string),
&m_out_conn_string_length, SQL_DRIVER_NOPROMPT);
EXPECT_EQ(SQL_SUCCESS, ret);
}

this seems to be connecting using admin:admin, but i've not looked into odbc code before, checking with others

@joshuali925
Copy link
Member

closing since CI doesn't use security enabled cluster

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request untriaged
Projects
None yet
Development

No branches or pull requests

2 participants