opensearch-project/sql-odbc

[2.12.0] Ensure compatibility for 2.12.0 default admin credentials changes

Closed this issue · 3 comments

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.

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.

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

closing since CI doesn't use security enabled cluster