simplesamlphp / simplesamlphp/simplesamlphp-module-metarefresh
RFE: metadata/metarefresh error handling improvements
Nadie ha tomado este issue todavía.
- Lenguaje dominante
- PHP
- Estrellas
- 9
- Forks
- 19
- Métricas de merge de PR
- Sin PR fusionados en 30 d
Descripción
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.
Guía de contribución
No hay ninguna guía de contribución indexada para este repositorio
Primeros pasos
- Lee el issue completo y luego la guía de contribución del proyecto.
- Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
- Haz un fork del repositorio y trabaja en una rama.
- Abre un pull request que haga referencia al número del issue.
Línea de trabajo
Comienza en la ruta de importación de Metarefresh y en el punto de entrada del manejo de errores de MetaLoader descrito en el issue. Traza cómo un mdui:Logo no válido afecta a una entidad y cómo la excepción capturada se convierte en el mensaje genérico de metadatos almacenados en caché. Se considera terminado cuando una entidad malformada no cancela la importación del feed y la excepción subyacente se conserva en el error informado.
Escrito por el modelo de indexación a partir del texto del issue.
Evaluación
- Stack tecnológico
- php
- Área
- backend
- Tipo de issue
- Nueva funcionalidad
- Dificultad
- 4/5
- Tiempo estimado
- 3-5 días
- Estado de actividad
- Estancado
- Claridad
- Bastante claro
- Aptitud para principiantes
- 35/100