Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

RelayState must not be immediately intellegible #127

Open
StefanoDevPro opened this issue Apr 26, 2021 · 8 comments
Open

RelayState must not be immediately intellegible #127

StefanoDevPro opened this issue Apr 26, 2021 · 8 comments
Labels
question Further information is requested

Comments

@StefanoDevPro
Copy link

StefanoDevPro commented Apr 26, 2021

Buongiorno a tutti, sto utilizzando la libreria spid-php-lib per lo sviluppo di Spid, ho un problema sulla risoluzione dell'errore riguardante il relayState: RelayState must not be immediately intellegible.
Potrei avere delle indicazioni su come risolverlo?

Grazie in anticipo

@StefanoDevPro
Copy link
Author

Nessuno è disponibile per la risoluzione di questo problema?

@umigliore
Copy link

Che parametro passi nel RelayState? dovresti mettere un identificativo non troppo in chiaro. Dice la documentazione tecnica: "Il Service Provider a tutela della privacy dell’utente nell’utilizzare questo parametro deve mettere in atto accorgimenti tali da rendere minima l’evidenza possibile sulla natura o tipologia della risorsa (servizio) richiesta"

@lorello
Copy link

lorello commented Apr 27, 2021

Noi abbiamo fatto un base64_encode del valore, lo abbiamo fatto nel nostro proxy di autenticazione, senza modificare la libreria.

@StefanoDevPro
Copy link
Author

Grazie, del suggerimento @lorello ho codificato anche io il base64, ora il test è superato.

Grazie

@donatissimo
Copy link

donatissimo commented May 3, 2021

perché invece di fare base64_encode che è comunque intellegibile, non si fa un md5 del RelayState?

@umigliore
Copy link

perché invece di fare base64_encode che è comunque intellegibile, non si fa un md5 del RelayState?

Il RelayState serve come valore per identificare la risposta, quindi si potrebbe usare anche un hash o anche un numero random (tipo otp) posto che sul tuo server hai modo di identificare facilmente a cosa si riferiva.

@donatissimo
Copy link

sì, è un valore opaco, dovrebbe funzionare come il bearer token di OAuth2. io adesso ci metto una stringa casuale però poi controllo che l'IDP mi ritorni lo stesso valore. Per farlo ho dovuto aggiungere del codice in spid-php-lib perché a prima vista non ho trovato controlli sul RelayState mandato indietro

@lussoluca
Copy link
Member

@donatissimo è codice che potrebbe essere incluso nella libreria? Puoi condividerlo? Se si, potresti aprire una MR? Grazie!

@lussoluca lussoluca added the question Further information is requested label Aug 29, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
question Further information is requested
Projects
None yet
Development

No branches or pull requests

5 participants