Idea #16314
closed
Support literal user/pass pairs in config file + make arvbox use it instead of SSO
Added by Peter Amstutz about 4 years ago. Updated over 3 years ago.
Description
Example config file excerpt:
Login:
# Authenticate test users only.
Test:
Enable: true
Users:
active:
email: active-user@arvados.local
password: xyzzy
projectviewer:
email: project-viewer@arvados.local
password: xyzzy
In the process, can we fix the hardcoded localhost/port check for redirect to SSO server in
lib/controller/handler_test.go line 197
lib/controller/rpc/conn_test.go line 61Related issues
Updated by Peter Amstutz about 4 years ago
- Story points set to 1.0
- Assigned To set to Peter Amstutz
Updated by Peter Amstutz about 4 years ago
- Target version changed from 2020-04-22 to 2020-05-06 Sprint
Updated by Peter Amstutz about 4 years ago
- Related to Idea #15322: Replace and delete sso-provider added
Updated by Peter Amstutz about 4 years ago
- Target version changed from 2020-05-06 Sprint to 2020-05-20 Sprint
Updated by Peter Amstutz almost 4 years ago
- Target version changed from 2020-05-20 Sprint to 2020-06-03 Sprint
Updated by Peter Amstutz almost 4 years ago
- Target version changed from 2020-06-03 Sprint to 2020-06-17 Sprint
Updated by Peter Amstutz almost 4 years ago
- Target version deleted (
2020-06-17 Sprint)
Updated by Tom Clegg almost 4 years ago
- Subject changed from arvbox uses PAM login instead of SSO to Support literal user/pass pairs in config file + make arvbox use it instead of SSO
Updated by Peter Amstutz almost 4 years ago
Maybe it should use hashed passwords at least?
Updated by Tom Clegg over 3 years ago
Since this is only intended for test environments, I don't think hashed passwords need to be a priority.
Updated by Peter Amstutz over 3 years ago
- Release set to 25
- Target version set to 2020-08-26 Sprint
Updated by Peter Amstutz over 3 years ago
- Assigned To changed from Peter Amstutz to Tom Clegg
Updated by
Updated by Tom Clegg over 3 years ago
This branch just adds config/controller support (planning to update arvbox / run-tests.sh in a separate branch).
16314-testuserdb @ 20cea53c48260a0cec3d588c2af520b27433b8c1 -- developer-run-tests: #2015
Updated by Lucas Di Pentima over 3 years ago
- File
lib/controller/localdb/login.go- Line 38 et al: Having so many
wantXcondition checks is making hard to follow the case statements, could we just check that the number of truewantXvars is 1 and then clean up the case statements for better readability?
- Line 38 et al: Having so many
- Tests failed -- adding the new config to export.go seem to be missing: developer-run-tests-remainder: #2103
/console
Updated by Tom Clegg over 3 years ago
Updated switch statement -- still a bit repetitive, but better -- and added missing config export entries.
16314-testuserdb @ 658f6be33e100b4cefd3ca7c332fe13970d6a245 -- developer-run-tests: #2016
Also added a wb2 branch, 16314-test-user-db @ arvados-workbench2|f47e1caaa5781e6f6d8e022f92cae34adf49eaad to add support for that config (I didn't change the wb2 test config, but we should probably do that when the API side is merged).
Updated by Lucas Di Pentima over 3 years ago
Updates at 658f6be33e100b4cefd3ca7c332fe13970d6a245 LGTM.
Workbench2 branch requires the following fix:
diff --git a/src/common/config.ts b/src/common/config.ts
index 26f8dddb..825bf132 100644
--- a/src/common/config.ts
+++ b/src/common/config.ts
@@ -235,7 +235,7 @@ export const mockClusterConfigJSON = (config: Partial<ClusterConfigJSON>): Clust
Enable: false,
},
Test: {
- Enable: boolean;
+ Enable: false,
}
},
Collections: {
I've manually tried arvbox and it works great!
Updated by Tom Clegg over 3 years ago
16314-arvbox-testuserdb @ b4091adb7ac1a85de6ae1f18895e9d8f9da5d441 -- developer-run-tests: #2033
Changes arvbox to use the Test login mechanism (with default logins user/user and admin/admin) instead of SSO. Uses a basic login form served from controller.
Updated by Lucas Di Pentima over 3 years ago
- File
doc/install/arvbox.html.textile.liquidmentions SSO related things on lines 83, 137, 140 - Workbench2 branch needs a fix at file
src/common/config.tsline 238 →s/;/,/ - Other than that, LGTM.
Updated by Tom Clegg over 3 years ago
- Target version changed from 2020-08-26 Sprint to 2020-09-09 Sprint
Updated by Tom Clegg over 3 years ago
- Status changed from In Progress to Resolved