vitam-ui icon indicating copy to clipboard operation
vitam-ui copied to clipboard

Bug #15176: change fields from required to requiredNotBlank

Open marob opened this issue 2 months ago • 1 comments

Description

La regex utilisée pour valider les identifiers (^[a-zA-Z0-9.\\-\\/+=@_ ]*$) ne devrait peut-être pas autoriser / et + car cela semble poser des problèmes (/ est interprété comme séparateur dans le path d'une URL, et + semble également poser problème

Type de changement

Indiquer le ou les types de changements

  • Build
  • PKI
  • Ansiblerie
  • Nouveau Code
  • Correction
  • Refactorisation de code
  • Autre

Documentation

Indiquer la documentation mise à jour

  • [ ] Quels sont les nouvelles documentations ?
  • [ ] Quels sont les modifications existantes ?
  • [ ] Quels sont les documentations ou sections de documentations supprimés ?

Tests

Indiquer comment le code à été testé (manuel, environnement, TU, etc)

  • manuel
  • environnement
  • TU

Migration

Indiquer si les modifications apportées impliquent une migration sur l'existant et comment la faire

Checklist

Sélectionner les éléments de la checklist

  • [ ] Mon code suit le style de code de ce projet.
  • [ ] J'ai commenté mon code, en particulier dans les classes et les méthodes difficile à comprendre.
  • [ ] J'ai fait les changements correspondant dans la documentation RAML.
  • [ ] J'ai fait les changements correspondant dans la documentation Métier.
  • [ ] J'ai fait les changements correspondant dans la documentation Technique.
  • [ ] J'ai rajouté les tests unitaires vérifiant mes fonctionnalités.
  • [ ] J'ai rajouté les tests de non régression vérifiant mes fonctionnalités.
  • [ ] Les tests unitaires nouveaux et existants passent avec succès localement.
  • [ ] Toutes les dépendances ont été mergées en priorité

Contributeur

Indiquer qui a développé cette fonctionnalité

  • VAS (Vitam Accessible en Service)
  • CEA (Commissariat à l'énergie atomique et aux énergies alternatives)

marob avatar Oct 07 '25 16:10 marob

Logo Checkmarx One – Scan Summary & Detailsdf188756-e696-459d-a898-9c00053a30c4

Great job! No new security vulnerabilities introduced in this pull request


Use @Checkmarx to reach out to us for assistance.

Just send a PR comment with @Checkmarx followed by a natural language request.

Examples: @Checkmarx how are you able to help me? @Checkmarx rescan this PR

vitam-prg avatar Oct 07 '25 16:10 vitam-prg