italia / spid-saml-check

Tool di verifica implementazione SPID SAML

European Union Public License 1.2

71 stars 59 forks source link

fix: url validation using built-in class #276

Open random42 opened 4 months ago

random42 commented 4 months ago

Uso la classe URL invece del RegExp, che è errato per alcune URL (tipo dentro docker http://web:4000/metadata, l'host non deve per forza avere il dominio).

random42 commented 4 months ago

275 Va a fixare questo

gabrielesilinic commented 1 month ago

@damikael per favore fai il merge di questa pull request. sto tentando di testare in localhost con il plugin di keycloak e sto avendo difficoltà a causa dello stato del regex. proabilmente non è l'unico problema che ho ma vorrei averne uno in meno.

ho guardato il file di utils e il cambiamento pare perfettamente ragionato. puoi anche testarlo aprendo i devtools.