AntoineVe/college-infos
1
0
Fork 0
Code Issues 1 Pull Requests Actions Packages Projects Releases 3 Wiki Activity

refactor(config): migrer RSS et Pronote vers l'endpoint commun et durcir le contrat #64

Merged
OpenCode merged 5 commits from refactor/issue-16-endpoints-rss-pronote into main 2026-09-13 16:40:15 +02:00
Conversation 0 Commits 5 Files Changed 15
+771 -191
OpenCode commented 2026-09-13 15:47:48 +02:00
Collaborator
Copy Link
Copy Source

Résumé

Reprend les migrations RSS (#60) et Pronote (#61) restées hors de main après une fusion empilée erronée, les rebase sur main (qui contient déjà #59 CalDAV et les correctifs sécurité #53/#54/#58) et applique les corrections du ticket #63.

  • Rebase de 097a27f (RSS) et ee89f13 (Pronote) sur main — aucun correctif sécurité de main n'est annulé (fichiers #53/#54/#58 byte-identiques).
  • Contrat ExternalEndpoint durci : schéma file retiré (réseau uniquement), URLs avec userinfo rejetées.
  • hide_input_in_errors=True sur PronoteSettings/CalDAVSettings/BlogSettings.
  • Fixtures file:// migrées vers https:// mocké (responses) ; matrice de tests paramétrée du contrat (précédence alias, loopback, extra/frozen, userinfo, redaction).
  • Suppression du contournement model_construct de l'alias CALDAV_URL : l'alias est désormais validé comme le champ canonique.
  • Retrait du support file:// de fetch_ical (branche morte) et de l'exemple résiduel du guide.
  • Documentation alignée : README.LLM.md, .env.example, GUIDE_DEV_PYTHON.md, docs/pronote-auth.md.

Validation

  • pytest --cov : 880 passés, couverture 95,17 %
  • ruff check . et ruff format --check . : OK
  • mypy . : Success (111 fichiers)
  • bandit -r pronote_sync/ : 0 constat
  • Revue indépendante, audit de sécurité et vérification : APPROVED / APPROVED / VERIFIED

Suivi

## Résumé Reprend les migrations RSS (#60) et Pronote (#61) restées hors de `main` après une fusion empilée erronée, les rebase sur `main` (qui contient déjà #59 CalDAV et les correctifs sécurité #53/#54/#58) et applique les corrections du ticket #63. - Rebase de `097a27f` (RSS) et `ee89f13` (Pronote) sur `main` — aucun correctif sécurité de `main` n'est annulé (fichiers #53/#54/#58 byte-identiques). - Contrat `ExternalEndpoint` durci : schéma `file` retiré (réseau uniquement), URLs avec userinfo rejetées. - `hide_input_in_errors=True` sur `PronoteSettings`/`CalDAVSettings`/`BlogSettings`. - Fixtures `file://` migrées vers `https://` mocké (`responses`) ; matrice de tests paramétrée du contrat (précédence alias, loopback, extra/frozen, userinfo, redaction). - Suppression du contournement `model_construct` de l'alias `CALDAV_URL` : l'alias est désormais validé comme le champ canonique. - Retrait du support `file://` de `fetch_ical` (branche morte) et de l'exemple résiduel du guide. - Documentation alignée : `README.LLM.md`, `.env.example`, `GUIDE_DEV_PYTHON.md`, `docs/pronote-auth.md`. ## Validation - `pytest --cov` : 880 passés, couverture 95,17 % - `ruff check .` et `ruff format --check .` : OK - `mypy .` : Success (111 fichiers) - `bandit -r pronote_sync/` : 0 constat - Revue indépendante, audit de sécurité et vérification : APPROVED / APPROVED / VERIFIED ## Suivi - Closes #16 (migration CalDAV + RSS + Pronote désormais complète dans `main`) - Closes #63 - Supersedes #61 (à fermer sans merge)
OpenCode added 5 commits 2026-09-13 15:47:48 +02:00
refactor(config): migrer le flux RSS vers l'endpoint commun c02f46245f 
Co-authored-by: Codex <codex@antoineve.me>
refactor(config): migrer les endpoints Pronote d1371cabbb 
Co-authored-by: Codex <codex@antoineve.me>
fix(config): refuser file:// et userinfo dans le contrat d'endpoint 177286b528 
Retire le schéma file du contrat ExternalEndpoint (CalDAV, Pronote iCal, Blog RSS) : seuls https et le loopback http CalDAV explicite restent acceptés. Rejette les URLs contenant userinfo. Ajoute hide_input_in_errors aux Settings. Réécrit les fixtures file:// en https mocké et ajoute une matrice de tests paramétrée du contrat.

Refs #63
docs: aligner la documentation sur le contrat ExternalEndpoint c89113ba36 
Documente le schéma *_ENDPOINT__URL unique, le statut déprécié des anciens noms et le refus de file:// et des credentials embarqués dans README.LLM.md, .env.example, GUIDE_DEV_PYTHON.md et docs/pronote-auth.md.

Refs #63
fix(config): valider l'alias CalDAV legacy et retirer file:// de fetch_ical 5a3e251ad6 
Supprime le contournement model_construct : l'alias CALDAV_URL est désormais validé comme le champ canonique (userinfo, host et port rejetés). Retire le support file:// de fetch_ical et corrige l'exemple résiduel du guide. Ajoute les tests négatifs de l'alias legacy et adapte le test de sécurité CalDAV au contrat durci.

Refs #63
OpenCode merged commit e5c14a9304 into main 2026-09-13 16:40:15 +02:00
Sign in to join this conversation.
Reviewers
No Reviewers
Labels
Clear labels
area:ai
Configuration et intégration de la synthèse IA
 area:auth
Authentification, session ou jeton Pronote
 area:caldav
Synchronisation et sérialisation CalDAV/iCalendar
 area:configuration
Configuration et validation des paramètres
 area:dependencies
Dépendances, forks et compatibilité externe
 area:documentation
Documentation utilisateur, développeur ou exploitation
 area:operations
Exploitation, supervision ou comportement du pipeline
 area:packaging
Packaging, métadonnées de distribution ou CI
 area:parsing
Parsing, normalisation ou déduplication des données
 area:pronote
Connexion ou données issues de Pronote
 area:rss
Récupération et acquittement des flux RSS
 area:security
Protection des secrets et durcissement de sécurité
 area:testing
Fixtures, couverture et stratégie de tests
 area:xmpp
Canal de notification XMPP
 bug
Anomalie ou comportement incorrect
 investigation
Investigation ou analyse de cause racine
 priority:critical
Traitement immédiat : impact direct sur la justesse du digest ou perte de données
 priority:high
Impact important : fonctionnalité bloquée, risque de perte ou de sécurité
 priority:low
Documentation, maintenance ou refactorisation non bloquante
 priority:medium
Amélioration significative après les urgences
No labels area:configuration area:pronote area:rss area:security area:testing
Milestone
No items
No Milestone
Projects
Clear projects
No projects
Assignees
Clear assignees
Codex (ChatGPT Codex) OpenCode AntoineVe (Antoine Van Elstraete)
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: AntoineVe/college-infos#64