Actions
Bug #17295
closed
arvados-server config should validate cluster ids
Status:
Resolved
Priority:
Normal
Assigned To:
Category:
API
Target version:
Story points:
-
Release:
Release relationship:
Auto
Description
User put in an invalid cluster id (6 characters). The controller behaves very strangely when you do that. arvadso-server should validate that the cluster id is exactly 5 alphanumeric (lowercase) characters.
This should check all occurrences of cluster ids in a file (Clusters, RemoteClusters, LoginCluster, etc).
Updated by Lucas Di Pentima about 3 years ago
- Target version changed from 2021-02-17 sprint to 2021-03-03 sprint
Updated by Lucas Di Pentima about 3 years ago
- Status changed from New to In Progress
Updated by Lucas Di Pentima about 3 years ago
Fix at a4ca691 - branch 17295-configured-cluster-ids-validation
Test run: developer-run-tests: #2335
- Adds cluster ID validation on config loader, with tests.
Updated by Peter Amstutz about 3 years ago
Lucas Di Pentima wrote:
Fix at a4ca691 - branch
17295-configured-cluster-ids-validation
Test run: developer-run-tests: #2335
- Adds cluster ID validation on config loader, with tests.
It occurs to me that you could have an all-numeric cluster id. I think that would behave correctly.
LGTM.
Updated by Anonymous about 3 years ago
- Status changed from In Progress to Resolved
Applied in changeset arvados|89a3090417e1f8241497a1d26425df9c5d2cb3ba.
Actions