search

italia / spid-keycloak-provider

Italian SPID authentication provider for Keycloak (https://www.keycloak.org/)
Apache License 2.0
62 stars 22 forks source link

Inserire il campo "Cessionario committente" nel metadata #27

Closed angelolamonaca closed 6 months ago

angelolamonaca commented 2 years ago

É possibile inserire il campo CessionarioCommittente nel metadata? In seguito alle nuove direttive (https://docs.italia.it/italia/spid/spid-regole-tecniche/it/stabile/metadata.html) é un campo obbligatorio per SP privati

mmariniopensoftware commented 2 years ago

Ciao @angelolamonaca, è probabile sia lo stesso problema segnalato in questa issue (https://github.com/italia/spid-keycloak-provider/issues/11)

In pratica il componente di @lscorcia non è aggiornato all'ultima specifica nel caso di un SP 'private' dato che è stato sviluppato per una PA . Io avevo lo stesso problema e ho aggiunto in maschera i campi per renderlo compatibile. Se hai urgenza puoi scaricarti il mio branch e provarlo, io al momento lo sto usando puntando sull'ambiente di demo, validazioni e login vanno a buon fine se può servirti provarle. Calcola che non è una PR ufficiale approvata dato che le modifiche sono un pò pesanti

Trovi il branch qui: (https://github.com/mmariniopensoftware/spid-keycloak-provider/tree/add-some-info-private-sp)

angelolamonaca commented 2 years ago

Ciao @mmariniopensoftware, complimenti per il lavoro svolto sul plugin, grazie mille!

lscorcia commented 2 years ago

Grazie @mmariniopensoftware , le modifiche sono ora integrate nella release 1.0.12 (https://github.com/italia/spid-keycloak-provider/releases/tag/1.0.12) tramite la PR #30. Attendo una conferma che tutto sia ok prima di procedere alla chiusura della issue. A presto e grazie ancora, ciao!

nicolabeghin commented 6 months ago

Chiudo in quanto già mergiato in https://github.com/italia/spid-keycloak-provider/pull/30