search

italia / spid-rails

SPID authentication for Ruby on Rails
MIT License
13 stars 9 forks source link

Validazioni su spid-saml-check/spid-validator #31

Closed Jeanluc74 closed 5 years ago

Jeanluc74 commented 5 years ago

Buongiorno,

sperando di non risultare troppo noioso accodo un'ulteriore questione. Ho appena terminato di effettuare i 104 test previsti dal tool in oggetto e purtroppo circa la metà delle validazioni non vanno a buon fine. La gem era stata testata anche con questo strumento? Ho visto che le validazioni vengono effettuate dalle classi /spid-0.19.0/lib/spid/saml2/*_validator.rb, ovviamente potrei integrare le validazioni mancanti, sarebbe comunque un lavoro non banale. In base alla vostra esperienza avete messo in pratica metodi più veloci?

Grazie mille come sempre.

Saluti,

Gian Luca

Jeanluc74 commented 5 years ago

Ho risolto il grosso delle validazioni intervenendo quasi esclusivamente su ResponseValidator, mancano all'appello 5 validazioni, di cui 3 riguardanti il confronto con il IssueInstant della Request (test 14, 39, 40 su spid-validator). Noto che ResponseValidator non ha visibilità su quel valore e quindi dovrei aggiungerlo al costruttore di Response come il request_uuid. Potete darmi una mano in questo senso?

Jeanluc74 commented 5 years ago

Con un po' di lavoro ho modificato il codice ancora e sono riuscito a chiudere le validazioni e a rifare i test daccapo. Se volete che contribuisca alla gem fatemi sapere.

davidlibrera commented 5 years ago

Ciao @Jeanluc74, scusa il ritardo nella risposta ma non avevo controllato la issue. Potresti aprire una pull request con le modifiche apportate? Almeno le mettiamo insieme. Domanda: il codice è relativo alla gemma spid-rails o a spid-ruby?