Skip to content

fix(iffi): HttpClient al posto di urllib con TLS disattivato - #14

Merged
Gabrymi93 merged 1 commit into
mainfrom
fix/iffi-http-client
Nov 6, 2026
Merged

Gabrymi93 merged 1 commit into
mainfrom
fix/iffi-http-client

Conversation

@Gabrymi93

Copy link
Copy Markdown
Member

Sintesi

Lo script IFFI disattivava la verifica TLS (CERT_NONE + check_hostname=False) per tutta la sessione, per aggirare la catena TLS incompleta di IdroGEO. Ora usa HttpClient (retry + fallback SSL confinato alla singola richiesta).

Contesto collegato

Audit cross-repo: rimozione dei CERT_NONE/verify=False nei repo di estrazione.

Cosa cambia

  • datasets/ispra-iffi/scripts/fetch_iffi.py: urllib + SSL context disattivato → HttpClient; errore HTTP esplicito con status.

Verifica

  • ruff check datasets/ispra-iffi/scripts/fetch_iffi.py → pulito
  • Chiamata reale: api_get("/iffi/italia") → dict valido dall'API IdroGEO.

Nessuna modifica a output CSV o pipeline.

Lo script disattivava verifica e hostname check per tutta la sessione
(CERT_NONE) per aggirare la catena TLS incompleta di IdroGEO. Ora usa
HttpClient (retry + fallback SSL confinato alla richiesta), con lo stesso
comportamento funzionale.
@Gabrymi93
Gabrymi93 merged commit dc59282 into main Nov 6, 2026
2 checks passed
@Gabrymi93
Gabrymi93 deleted the fix/iffi-http-client branch November 6, 2026 21:31
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