Skip to content

fix(probe): user_agent dal config passato a probe_url - #428

Merged
Gabrymi93 merged 1 commit into
mainfrom
fix/probe-user-agent
Oct 16, 2026
Merged

Gabrymi93 merged 1 commit into
mainfrom
fix/probe-user-agent

Conversation

@Gabrymi93

Copy link
Copy Markdown
Member

Sintesi

Il probe iniziale di so-catalog non riceveva il user_agent dal config [tool.observatory] — il WAF ANAC bloccava il probe con 403.

Causa

CI failure su appalti-pubblici#9: 403 Forbidden sul probe di dati.anticorruzione.it. Il collector CKAN leggeva gia' user_agent dal config, ma probe_url usava UA di default.

Fix

  • probe_url(base_url, user_agent=None) — parametro opzionale
  • _probe_once(base_url, user_agent=None) — passato al HttpClient
  • build_catalog passa cfg.get(user_agent) a probe_url

Verifiche

  • 34 test verdi (build + radar_check)
  • Live su appalti-pubblici: radar GREEN (prima 403/YELLOW)

Nota

open-mur e' un caso diverso: dati-ustat.mur.gov.it non e' raggiungibile dai runner GHA (timeout). Non e' un bug del codice — e' la fonte che blocca i runner. Da gestire con timeout-minutes sul workflow o accettare che某些 fonti non funzionano in CI.

Il WAF ANAC blocca il probe iniziale di so-catalog (UA di default).
probe_url e _probe_once accettano ora user_agent opzionale, passato dal
config [tool.observatory] nel build_catalog.

Verificato su appalti-pubblici: radar GREEN (prima 403/YELLOW).
Risolve il CI failure di appalti-pubblici#9.
@Gabrymi93
Gabrymi93 merged commit 3898331 into main Oct 16, 2026
1 check passed
@Gabrymi93
Gabrymi93 deleted the fix/probe-user-agent branch October 16, 2026 15:09
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