fix(security) : ignorer les placeholders URL documentaires #36

Open
Codex wants to merge 1 commits from fix/issue-34-secret-placeholder into main
Collaborator

Résumé

Corrige le faux positif de scripts/check_secrets.py sur les URLs d'exemple masquées de docs/pronote-auth.md.

Décision d'implémentation

  • Les paramètres sensibles d'URL continuent d'être signalés par défaut.
  • Seuls les marqueurs documentaires explicites et structurels ({...}, <...>, ..., points/bullets de masquage) sont ignorés.
  • Une valeur réelle reste détectée et sa valeur n'est jamais imprimée.
  • Les tests couvrent les quatre formes de placeholders et la détection d'une valeur réelle.

Validation

  • 10 passed : tests/unit/test_check_secrets.py
  • scripts/check_secrets.py retourne 0 sur le dépôt courant
  • ruff check et ruff format --check
  • Pre-commit complet : Ruff, format, mypy, Bandit, conflits, fichiers, detect-secrets
  • Aucun secret réel ni accès réseau utilisé

Closes #34

## Résumé Corrige le faux positif de `scripts/check_secrets.py` sur les URLs d'exemple masquées de `docs/pronote-auth.md`. ## Décision d'implémentation - Les paramètres sensibles d'URL continuent d'être signalés par défaut. - Seuls les marqueurs documentaires explicites et structurels (`{...}`, `<...>`, `...`, points/bullets de masquage) sont ignorés. - Une valeur réelle reste détectée et sa valeur n'est jamais imprimée. - Les tests couvrent les quatre formes de placeholders et la détection d'une valeur réelle. ## Validation - `10 passed` : `tests/unit/test_check_secrets.py` - `scripts/check_secrets.py` retourne 0 sur le dépôt courant - `ruff check` et `ruff format --check` - Pre-commit complet : Ruff, format, mypy, Bandit, conflits, fichiers, detect-secrets - Aucun secret réel ni accès réseau utilisé Closes #34
Codex added the bugpriority:higharea:securityarea:documentation labels 2026-09-12 15:18:27 +02:00
Codex added 1 commit 2026-09-12 15:18:28 +02:00
You are not authorized to merge this pull request.
This pull request can be merged automatically.
This branch is out-of-date with the base branch
View command line instructions

Checkout

From your project repository, check out a new branch and test the changes.
git fetch -u origin fix/issue-34-secret-placeholder:fix/issue-34-secret-placeholder
git checkout fix/issue-34-secret-placeholder
Sign in to join this conversation.
No Reviewers
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: AntoineVe/college-infos#36