Skip to content

Commit

Permalink
Manual backport 2.x with password validation related fixes and versio…
Browse files Browse the repository at this point in the history
…n bump to 2.8.0 (opensearch-project#1437)

* Change the testuser's password in some integration test cases into a stronger password (opensearch-project#1428)

* Change the testuser's password into a stronger password

Signed-off-by: Ryan Liang <[email protected]>

* Bump the version to 2.8.0

Signed-off-by: Ryan Liang <[email protected]>

---------

Signed-off-by: Ryan Liang <[email protected]>
  • Loading branch information
RyanL1997 authored May 25, 2023
1 parent 8579406 commit 8fe290b
Show file tree
Hide file tree
Showing 7 changed files with 15 additions and 15 deletions.
4 changes: 2 additions & 2 deletions .github/workflows/cypress-test.yml
Original file line number Diff line number Diff line change
Expand Up @@ -9,9 +9,9 @@ env:
START_CMD: 'node ../scripts/opensearch_dashboards --dev --no-base-path --no-watch --opensearch_security.multitenancy.enable_aggregation_view=true'
OPENSEARCH_SNAPSHOT_CMD: 'node ../scripts/opensearch snapshot'
SPEC: 'cypress/integration/plugins/security-dashboards-plugin/aggregation_view.js,'
OPENSEARCH_VERSION: 2.7.0
OPENSEARCH_VERSION: 2.8.0
PLUGIN_NAME: opensearch-security
PLUGIN_VERSION: 2.7.0.0
PLUGIN_VERSION: 2.8.0.0

jobs:
tests:
Expand Down
4 changes: 2 additions & 2 deletions .github/workflows/integration-test.yml
Original file line number Diff line number Diff line change
Expand Up @@ -5,9 +5,9 @@ on: [push, pull_request]
env:
TEST_BROWSER_HEADLESS: 1
CI: 1
OPENSEARCH_VERSION: 2.7.0
OPENSEARCH_VERSION: 2.8.0
PLUGIN_NAME: opensearch-security
PLUGIN_VERSION: 2.7.0.0
PLUGIN_VERSION: 2.8.0.0

jobs:
tests:
Expand Down
4 changes: 2 additions & 2 deletions .github/workflows/prerequisite-test.yml
Original file line number Diff line number Diff line change
Expand Up @@ -9,6 +9,6 @@ jobs:
steps:
- name: Check for the existence of the OpenSearch Security Plugin artifact
env:
opensearch_version: 2.5.0
opensearch_security_plugin_version: 2.5.0.0
opensearch_version: 2.8.0
opensearch_security_plugin_version: 2.8.0.0
run: wget -S --spider https://ci.opensearch.org/ci/dbc/distribution-build-opensearch/${opensearch_version}/latest/linux/x64/tar/builds/opensearch/plugins/opensearch-security-${opensearch_security_plugin_version}.zip || (echo "Please make sure security plugin has been bumped to the same version and added to manifest." && exit 1)
4 changes: 2 additions & 2 deletions opensearch_dashboards.json
Original file line number Diff line number Diff line change
@@ -1,7 +1,7 @@
{
"id": "securityDashboards",
"version": "2.7.0.0",
"opensearchDashboardsVersion": "2.7.0",
"version": "2.8.0.0",
"opensearchDashboardsVersion": "2.8.0",
"configPath": ["opensearch_security"],
"requiredPlugins": ["navigation", "savedObjectsManagement"],
"server": true,
Expand Down
6 changes: 3 additions & 3 deletions package.json
Original file line number Diff line number Diff line change
@@ -1,10 +1,10 @@
{
"name": "opensearch-security-dashboards",
"version": "2.7.0.0",
"version": "2.8.0.0",
"main": "target/plugins/opensearch_security_dashboards",
"opensearchDashboards": {
"version": "2.7.0",
"templateVersion": "2.7.0"
"version": "2.8.0",
"templateVersion": "2.8.0"
},
"license": "Apache-2.0",
"homepage": "https://github.com/opensearch-project/security-dashboards-plugin",
Expand Down
2 changes: 1 addition & 1 deletion test/jest_integration/multi_tenancy.test.ts
Original file line number Diff line number Diff line change
Expand Up @@ -67,7 +67,7 @@ describe('start OpenSearch Dashboards server', () => {

async function createTestUser(username: string = undefined, password: string = undefined) {
const testUserName = username || `test_user_${Date.now()}`;
const testUserPassword = password || 'Test_123';
const testUserPassword = password || 'testUserPassword123';

await createOrUpdateEntityAsAdmin(root, 'internalusers', testUserName, {
password: testUserPassword,
Expand Down
6 changes: 3 additions & 3 deletions test/jest_integration/security_entity_api.test.ts
Original file line number Diff line number Diff line change
Expand Up @@ -68,7 +68,7 @@ describe('start OpenSearch Dashboards server', () => {

it('create/get/update/list/delete internal user', async () => {
const testUsername = `test_user_${Date.now()}`;
const testUserPassword = 'Test_123';
const testUserPassword = 'testUserPassword123';

const createUserResponse = await createOrUpdateEntityAsAdmin(
root,
Expand Down Expand Up @@ -270,7 +270,7 @@ describe('start OpenSearch Dashboards server', () => {

it('get account info', async () => {
const testUsername = `test_user_${Date.now()}`;
const testUserPassword = 'Test_123';
const testUserPassword = 'testUserPassword123';

await createOrUpdateEntityAsAdmin(root, 'internalusers', testUsername, {
description: 'test user description',
Expand All @@ -297,7 +297,7 @@ describe('start OpenSearch Dashboards server', () => {

it('self reset password as non-admin', async () => {
const testUsername = `test_user_${Date.now()}`;
const testUserPassword = 'Test_123';
const testUserPassword = 'testUserPassword123';
const readAllRole = 'read_all'; // use predefined "read_all" role

await createOrUpdateEntityAsAdmin(root, 'internalusers', testUsername, {
Expand Down

0 comments on commit 8fe290b

Please sign in to comment.