Skip to content

[patch] Add validation for customization archive name in interactive installation#2230

Open
PalVala wants to merge 5 commits into
masterfrom
mascore-13554
Open

[patch] Add validation for customization archive name in interactive installation#2230
PalVala wants to merge 5 commits into
masterfrom
mascore-13554

Conversation

@PalVala
Copy link
Copy Markdown

@PalVala PalVala commented Apr 30, 2026

Problem

  • Installation fails when customization archive names contain spaces or invalid characters, causing this Kubernetes API error:

    Invalid value: "Custom Archive": spec.settings.customizationList[0].customizationArchiveName
    in body should match '^[0-9a-zA-Z][0-9a-zA-Z-_.]+$'

Solution

  • Added client-side validation to catch invalid names early during interactive installation.

Changes:

  • Added CustomizationArchiveNameValidator class in python/src/mas/cli/validators.py
  • Applied validator to customization archive name prompt in python/src/mas/cli/install/settings/manageSettings.py
  • Validation Rules
  • Must start with alphanumeric character (a-z, A-Z, 0-9)
  • Can contain: alphanumeric, hyphens (-), underscores (_), periods (.)
  • No spaces allowed

User Experience

  • Invalid input is rejected immediately with a clear error message and automatic re-prompt:

    Customization archive name: CustomMas
    Invalid format. Must start with alphanumeric character and contain only
    alphanumeric characters, hyphens (-), underscores (
    ), and periods (.). No spaces allowed.
    Customization archive name: CustomMas

Files Changed:

  • python/src/mas/cli/validators.py
  • python/src/mas/cli/install/settings/manageSettings.py

Screenshots

Screenshot 2026-04-27 at 11 12 02 PM Screenshot 2026-04-27 at 11 12 14 PM

@PalVala PalVala requested a review from a team as a code owner April 30, 2026 07:00
@PalVala PalVala changed the title [patch] Mascore 13554 : Add validation for customization archive name in interactive installation [patch] Add validation for customization archive name in interactive installation Apr 30, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants