Supporto alla versione 22 o superiore

italia / spid-keycloak-provider

Italian SPID authentication provider for Keycloak (https://www.keycloak.org/)

Apache License 2.0

62 stars 22 forks source link

Supporto alla versione 22 o superiore #57

Closed pingrami closed 6 months ago

pingrami commented 11 months ago

E' previsto il supporto alla versione 22 (o superiore) di Keycloak e relativo adeguamento alla nuova interfaccia 'keyclaok.v2'?

lucrus73 commented 10 months ago

duplicato di https://github.com/italia/spid-keycloak-provider/issues/52

nicolabeghin commented 9 months ago

PR sottomessa per compatibilità Keycloak 23 (no interfaccia grafica) https://github.com/italia/spid-keycloak-provider/pull/59

nicolabeghin commented 8 months ago

Per chi avesse voglia/tempo di aiutare: qui https://github.com/italia/spid-keycloak-provider/pull/59 l'inizio dell'adattamento della UI del provider basato su ConfiguredProvider come richiesto da Keycloak 23.x

Ettore-Morasso commented 7 months ago

Ciao, innanzitutto complimenti ai contributori per l'ottimo lavoro svolto su questa estensione, tuttavia come altri leggendo mi trovo nella stessa situazione di incompatibilità dell'estensione con le più recenti versioni di keycloak. Vorrei chiedere a tal proposito se esiste una roadmap per il rilascio della nuova estensione o, più semplicemente una stima sui tempi necessari. Grazie.

nicolabeghin commented 7 months ago

@Ettore-Morasso il plugin dovrebbe funzionare senza problemi, il problema è la sua configurazione se fatta da UI, dato che da standard dalle ultime versioni non è possibile estendere l'interfaccia di configurazione di un IdP custom. Se usi il plugin senza configurarlo da web ma solo tramite REST (es. keycloak-spid-provider-configuration-client), dovrebbe andare tutto ok.

Per il resto qui la discussione sul repo ufficiale Keycloak https://github.com/keycloak/keycloak/discussions/25862

Ettore-Morasso commented 7 months ago

Grazie @nicolabeghin , confermo che il plugin funziona correttamente sulla versione 23.0.5, al netto dei problemi con la ui. Il progetto del configuratore rest anch'esso funzione egregiamente anche se alcune chiamate e alcuni valori default nel file di configurazione devono essere adeguati ai nuovi endpoint di keycloak. Grazie per il lavoro svolto.

pingrami commented 7 months ago

Ciao @Ettore-Morasso, sapresti indicarmi che versione del plugin hai utilizzato?

Ettore-Morasso commented 7 months ago

Ciao @pingrami , devi usare il repo git di Nicola, branch "upgrade_keycloak_23", lo trovi qui: https://github.com/nicolabeghin/keycloak-spid-provider/tree/upgrade_keycloak_23 fai la build come al solito e poi se guardi la risposta che mi ha dato Nicola vedi che c'è un link al tool di configurazione per i vari idp spid....c'è qualcosina da sistemare lì ma funziona più che bene. Saluti.

nicolabeghin commented 7 months ago

link al tool di configurazione per i vari idp spid....c'è qualcosina da sistemare lì ma funziona più che bene.

@Ettore-Morasso sarebbe utile se potessi indicare cosa vada aggiornato nel client o direttamente sottomettere una PR - grazie!

@pingrami a breve farò il merge del branch in oggetto così da creare il JAR pronto all'uso

nicolabeghin commented 7 months ago

1.0.16 rilasciata https://github.com/nicolabeghin/keycloak-spid-provider/releases/tag/1.0.16 con JAR pronto all'uso

Ho da pochissimo avuto anche i permessi di maintainer sul repo italia/spid-keycloak-provider e provvederò ad allineare

Ettore-Morasso commented 7 months ago

Ciao Nicola, a breve farò una pr e valuterai. Grazie

Ettore-Morasso commented 7 months ago

Ciao Nicola, il codice sarebbe pronto ma chiaramente non posso fare commit e creare la pr su questo repo non avendovi alcun diritto, quindi come vogliamo fare? crei un branch su cui mi dai accesso, o semplicemente ti invio i file?

nicolabeghin commented 7 months ago

@Ettore-Morasso su GitHub funziona che

fai il fork del repository -> ti crea qualcosa tipo ettore-morasso/keycloak-spid-provider-configuration-client
committi quanto devi in ettore-morasso/keycloak-spid-provider-configuration-client
sottometti PR dal tuo repo a quello originario

Ettore-Morasso commented 6 months ago

Ciao Nicola, la pull request è stata fatta ( https://github.com/nicolabeghin/keycloak-spid-provider-configuration-client/pull/5 ). Saluti

nicolabeghin commented 6 months ago

Ciao Nicola, la pull request è stata fatta ( nicolabeghin/keycloak-spid-provider-configuration-client#5 ). Saluti

aggiunti alcuni commenti prima di fare il merge, grazie!

nicolabeghin commented 6 months ago

Chiudo issue in quanto conclusa con PR https://github.com/italia/spid-keycloak-provider/pull/59