ncarandini
commented
6 years ago @marcoz-tsn ho incorporato le correzioni da te proposte relative ai path delle immagini. In generale ti suggerisco di aprire una issue per ogni specifico problema e di proporre una PR per quel singolo problema. Se si mischiano cose diverse in un'unica PR, non è possibile utilizzarne una sola parte ed è un po' come prendere tutto il pacchetto o lasciare. Rimane dunque aperta la questione delle modifiche da te proposte per far funzionare questa libreria col server di test di SAML (saml-testenv2) ma come detto devo verificare se è un problema della libreria, che pare funzionare bene con gli altri server di IP e non un problema del server di test stesso. Per quanto detto, chiudo la tua richiesta senza accettare la PR.
marcoz-tsn
Ciao, ho visto la tua PR ed è interessante, devo solo verificare prima di accettarla che non crei problemi con gli altri IP. In generale ti pregherei, prima di proporre una PR, di creare una issue in cui descrivi il problema e nella PR cosa proponi per risolvere la issue.