Skip to content

Hygiène scrapers : curl -f, imports morts, User-Agent, .gitignore#9

Open
hermias1 wants to merge 1 commit into
Dahliyaal:mainfrom
hermias1:fix/scrape-hygiene
Open

Hygiène scrapers : curl -f, imports morts, User-Agent, .gitignore#9
hermias1 wants to merge 1 commit into
Dahliyaal:mainfrom
hermias1:fix/scrape-hygiene

Conversation

@hermias1

Copy link
Copy Markdown

Micro-fixes issus de l'audit des scrapers — aucun changement de comportement fonctionnel, tout est autoportant.

  • update_dila.sh : ajoute -f aux 3 curl de téléchargement. Sans lui, une réponse HTTP 4xx/5xx est enregistrée telle quelle comme fichier .tar.gz, puis tar xzf échoue sur une page d'erreur. Aligne le comportement sur les autres scripts qui utilisent déjà curl -sf. (Les curl de listing restent en -s : leur échec produit un LATEST vide déjà géré par le garde if.)
  • scrape_ariane.py : retire import os inutilisé.
  • rescrape_cedh.py : retire from urllib.parse import quote inutilisé.
  • scrape_cedh_gaps.py : User-Agent aligné sur les autres scrapers (... contact: dahliyaal@justicelibre.org) au lieu de gap-filler sans contact — politesse cohérente envers HUDOC.
  • .gitignore : ajoute .DS_Store.

Vérifié : py_compile sur les 3 fichiers Python, bash -n update_dila.sh.

À noter (hors périmètre, non touché ici) : export_piste.py est du code mort superseded par export_piste_v2.py — suppression laissée à votre appréciation.

Micro-fixes issus de l'audit des scrapers, sans changement de comportement :
- update_dila.sh : ajoute -f aux curl de téléchargement. Sans lui, une
  réponse HTTP 4xx/5xx est enregistrée comme fichier, puis tar échoue —
  incohérent avec les autres scripts qui utilisent déjà curl -sf.
- scrape_ariane.py : retire 'import os' inutilisé.
- rescrape_cedh.py : retire 'from urllib.parse import quote' inutilisé.
- scrape_cedh_gaps.py : User-Agent aligné sur les autres scrapers (avec
  contact) au lieu de 'gap-filler' sans contact.
- .gitignore : ajoute .DS_Store.
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.

1 participant