Skip to content

Conversation

@marob
Copy link
Contributor

@marob marob commented Oct 7, 2025

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 marob added this to the IT 159 milestone Oct 7, 2025
@marob marob added the bug Something isn't working label Oct 7, 2025
@vitam-prg
Copy link
Collaborator

vitam-prg commented Oct 7, 2025

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

Copy link
Contributor

@ebernard ebernard left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Bug non critique, non bloquant, et non bloquant release 9.0. À ne pas merger pour l'instant.

@marob marob force-pushed the bug-15176 branch 4 times, most recently from d470c66 to c920056 Compare October 20, 2025 12:48
@ebernard ebernard dismissed their stale review October 30, 2025 09:08

Cela peut être mergé maintenant que la nouvelle branche master_9.0.x a été créée. Il faudra bien penser à backporter

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants