simplesamlphp / simplesamlphp/simplesamlphp-module-metarefresh

RFE: metadata/metarefresh error handling improvements

Aperta
#1 3 commenti 0 reazioni 0 assegnatari Vedi su GitHub

Nessuno ha ancora preso questa issue.

Lingua principale
PHP
Stelle
9
Fork
19
Metriche di merge delle PR
Nessuna PR unita negli ultimi 30g

Descrizione

Currently the eduGAIN feed contains an SP with empty <mdui:Logo>. When running Metarefresh, this makes mdui\Logo throw an exception, saying Missing url value for Logo. Now that's okay, although it doesn't tell me anything about which entity it is.

But 1) the exception propagates all the way upwards, cancelling the import of the entire feed – meaning that just about any SP anywhere can accidentally break my whole eduGAIN access. (Somehow even eduGAIN didn't catch this error.) Maybe it would be possible to skip just that one entity – or even load it without any logos, as they're probably not crucial to the operation.

And 2) the exception is eventually swallowed and thrown away in MetaLoader, which instead shows a very generic "XML parser error when parsing https://fedi.litnet.lt/metadata/edugain.xml - attempting to re-use cached metadata". I spent quite some time trying to figure out that one, as the file is perfectly valid XML. I think it would be good to show the actual exception that was caught.

Guida per i contributori

Nessuna guida per i contributori indicizzata per questo repository

Come iniziare

  1. Leggi tutta la issue e poi la guida ai contributi del progetto.
  2. Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
  3. Fai un fork del repository e lavora su un branch.
  4. Apri una pull request che faccia riferimento al numero della issue.

Direzione di ricerca

Parti dal percorso di importazione di Metarefresh e dal punto di ingresso della gestione degli errori di MetaLoader descritto nell’issue. Traccia il modo in cui un mdui:Logo non valido influisce su un’entità e il modo in cui l’eccezione intercettata diventa il messaggio generico sui metadati memorizzati nella cache. Il lavoro è completato quando un’entità malformata non annulla l’importazione del feed e l’eccezione sottostante viene conservata nell’errore segnalato.

Scritto dal modello di indicizzazione a partire dal testo della issue.

Valutazione

Stack tecnologico
php
Ambito
backend
Tipo di issue
Funzionalità
Difficoltà
4/5
Tempo stimato
3-5 giorni
Stato di attività
Ferma
Chiarezza
Abbastanza chiara
Idoneità per principianti
35/100

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.