Project

General

Profile

Actions

Bug #17295

closed

arvados-server config should validate cluster ids

Added by Peter Amstutz almost 4 years ago. Updated almost 4 years ago.

Status:
Resolved
Priority:
Normal
Assigned To:
Category:
API
Target version:
Start date:
02/19/2021
Due date:
% Done:

100%

Estimated time:
(Total: 0.00 h)
Story points:
-
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).


Subtasks 1 (0 open1 closed)

Task #17323: Review 17295-configured-cluster-ids-validationResolvedPeter Amstutz02/19/2021

Actions
Actions #1

Updated by Peter Amstutz almost 4 years ago

  • Description updated (diff)
Actions #2

Updated by Peter Amstutz almost 4 years ago

  • Description updated (diff)
Actions #3

Updated by Peter Amstutz almost 4 years ago

  • Description updated (diff)
Actions #4

Updated by Peter Amstutz almost 4 years ago

  • Assigned To set to Lucas Di Pentima
Actions #5

Updated by Lucas Di Pentima almost 4 years ago

  • Target version changed from 2021-02-17 sprint to 2021-03-03 sprint
Actions #6

Updated by Lucas Di Pentima almost 4 years ago

  • Status changed from New to In Progress
Actions #7

Updated by Lucas Di Pentima almost 4 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.
Actions #8

Updated by Peter Amstutz almost 4 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.

Actions #9

Updated by Anonymous almost 4 years ago

  • Status changed from In Progress to Resolved
Actions #10

Updated by Peter Amstutz almost 4 years ago

  • Release set to 37
Actions

Also available in: Atom PDF