simplesamlphp / simplesamlphp/simplesamlphp-module-metarefresh
RFE: metadata/metarefresh error handling improvements
Dieses Issue hat noch niemand übernommen.
- Vorherrschende Sprache
- PHP
- Sterne
- 9
- Forks
- 19
- PR-Merge-Kennzahlen
- Keine gemergten PRs in 30 T.
Beschreibung
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.
Beitragsleitfaden
Für dieses Repository ist kein Beitragsleitfaden indexiert
Erste Schritte
- Lies das ganze Issue und danach den Beitragsleitfaden des Projekts.
- Schreib ins Issue, dass du es übernimmst — das erspart doppelte Arbeit.
- Forke das Repository und arbeite in einem Branch.
- Öffne einen Pull Request, der die Issue-Nummer nennt.
Rechercherichtung
Beginnen Sie beim Metarefresh-Importpfad und beim im Issue beschriebenen Einstiegspunkt für die Fehlerbehandlung von MetaLoader. Verfolgen Sie, wie ein ungültiges mdui:Logo eine Entität beeinflusst und wie die abgefangene Exception zur generischen Meldung für zwischengespeicherte Metadaten wird. Als erledigt gilt die Aufgabe, wenn eine fehlerhafte Entität den Feed-Import nicht abbricht und die zugrunde liegende Exception im gemeldeten Fehler erhalten bleibt.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Bewertung
- Tech-Stack
- php
- Bereich
- backend
- Issue-Typ
- Feature
- Schwierigkeit
- 4/5
- Geschätzter Aufwand
- 3-5 Tage
- Aktivitätsstatus
- Veraltet
- Klarheit
- Größtenteils klar
- Anfängerfreundlichkeit
- 35/100