Add own exceptions to the library
Nessuno ha ancora preso questa issue.
Valutazione
- Difficoltà
- 5/5
- Tempo stimato
- Più di una settimana
- Idoneità per principianti
- 25/100
Direzione di ricerca
Iniziate esaminando i punti di eccezione indicati in \SAML2\SOAP::receive() e \SAML2\Binding::getCurrentBinding(), quindi analizzate le altre semplici eccezioni della libreria. Definite l'ambito delle eccezioni personalizzate e come le eccezioni provenienti da SimpleSAMLphp debbano essere trasferite in questa libreria. Il lavoro è completo quando gli errori rilevanti utilizzano eccezioni specifiche che i chiamanti possono intercettare e presentare con messaggi tradotti e significativi.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Descrizione
While trying to add more meaningful messages to some of the typical errors, I've noticed that at some points in the library we are throwing simple exceptions with a message that's not really descriptive of the real problem, nor even translated. For instance, in \SAML2\SOAP::receive():
$postText = file_get_contents('php://input');
if (empty($postText)) {
throw new Exception('Invalid message received to AssertionConsumerService endpoint.');
}
or in \SAML2\Binding::getCurrentBinding():
throw new Exception('Unable to find the current binding.');
If these were custom exceptions, we could capture them easily and display a meaningful, translated error message to the user, depending on where we are. Therefore I think we should even move some of the exceptions in SimpleSAMLphp to this library, and extend them with specific ones.
- Lingua principale
- PHP
- Stelle
- 306
- Fork
- 138
- Metriche di merge delle PR
- Nessuna PR unita negli ultimi 30g
Guida per i contributori
Nessuna guida per i contributori indicizzata per questo repository
Come iniziare
- Leggi tutta la issue e poi la guida ai contributi del progetto.
- Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
- Fai un fork del repository e lavora su un branch.
- Apri una pull request che faccia riferimento al numero della issue.
Altre issue di simplesamlphp/saml2
-
Difficoltà 3/5 1-2 giorni Idoneità per principianti 55/100
simplesamlphp/saml2#408 · 4 commenti ·
-
Difficoltà 3/5 1-2 giorni Idoneità per principianti 35/100
simplesamlphp/saml2#397 ·
-
Difficoltà 4/5 3-5 giorni Idoneità per principianti 35/100
simplesamlphp/saml2#360 ·
-
Difficoltà 3/5 1-2 giorni Idoneità per principianti 38/100
simplesamlphp/saml2#345 · 1 commento ·
-
Difficoltà 4/5 3-5 giorni Idoneità per principianti 35/100
simplesamlphp/saml2#309 · 18 commenti ·
Tutte le issue di simplesamlphp/saml2
Issue simili
-
priority: p3
Difficoltà 2/5 1-3 ore Idoneità per principianti 72/100
googleapis/librarian#7636 ·
-
0. Needs triage bug
Difficoltà 2/5 1-3 ore Idoneità per principianti 78/100
nextcloud/fulltextsearch#1011 ·
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 84/100
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 78/100
phpstan/phpstan-doctrine#794 ·
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 74/100
Automattic/static-site-importer#1767 ·