spid-laravel
spid-laravel copied to clipboard
italia
adding support for php 8.0 and php-saml 4.0
Salve.
Da quel che ho visto php-saml 3.4.1 è l'unica dipendenza che imponga di utilizzare php 7, ma è già disponibile la versione 4.0 di php-saml che introduce il supporto a php 8.0. Ho provato ad adattare la patch per nuova versione di php-saml ma al momento del login quando seleziono un identity provider da solo un errore 419 page expired.
Per caso c'è un modo con cui possa ottenere qualche ulteriore informazione riguardo all'erroe per fare debug?
Grazie mille, Riccardo.
Ciao @Rickrk4, innanzitutto grazie per il tuo contributo. Rispetto all'errore non saprei darti molte indicazioni se non un sospetto relativo a qualcosa che non va nel recupero della sessione dopo la request. Se magari metti la patch e le modifiche che hai fatto in una PR, possiamo dare un'occhiata insieme.
In alternativa, anziché continuare con le patch a php-saml, si potrebbe andare nella direzione di integrare https://github.com/italia/spid-php che è un progetto molto più maturo e aggiornato. In questo caso lo sforzo sarebbe quello di sostituire php-saml con spid-php in modo trasparente mantenendo la stessa API attuale di spid-laravel (possiamo valutare poi anche eventuali breaking changes).
Ciao @pdavide, grazie per l'aiuto. Sto lavorando su un fork del repository https://github.com/Rickrk4/spid-laravel, ho inserito le modifiche nella branch testing, e questa è la patch che ho adattato io https://github.com/Rickrk4/spid-laravel/blob/testing/patches/php-saml-4.0.0-spid.patch . Se non ho capito male per adesso il progetto più mantenuto sarebbe spid-php, e quindi anche in vista del futuro converrebbe integrarlo nell'api per laravel al posto di php-saml come da te proposto, giusto?
Grazie per
Ciao @Rickrk4, si corretto. Converrebbe passare a spid-php al posto di php-saml.
Ogni volta che aggiorno un pacchetto con composer mi appare questo messaggio:
italia/spid-laravel[v1.2.0-beta, ..., v1.2.6-beta] require php ^7.2.0 -> your php version (8.0.2) does not satisfy that requirement.
È possibile fare qualcosa?
Ciao @cod3rshotout, forse @Rickrk4 potrebbe esserti d'aiuto con la soluzione che aveva iniziato a predisporre. Se poi la soluzione trovata è stabile e testata si potrebbe anche integrarla qui con una PR.
@pdavide Come mai questo progetto non è più aggiornato? È l'unico package spid per laravel ma sembra essere abbandonato
Ciao @cod3rshotout, il progetto non è abbandonato. Semplicemente non ci sono stati molti contributi da parte della community se non la #81 di un paio di settimane fa.
Oggi ho smanettato per un upgrade a Laravel 9 (che richiede PHP 8) e non mi risulta che onelogin/php-saml 3.4.1 abbia una dipendenza stretta a PHP 7. L'ho provato empiricamente e faccio il login SPID senza particolari problemi.
Certo, l'adozione a spid-php sarebbe ulteriormente preferibile.
