Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

upgrade to spid-php-lib v0.20 #6

Open
simevo opened this issue Oct 13, 2018 · 2 comments
Open

upgrade to spid-php-lib v0.20 #6

simevo opened this issue Oct 13, 2018 · 2 comments
Assignees

Comments

@simevo
Copy link
Owner

simevo commented Oct 13, 2018

dopo che spid-symfony-bundle è passato a ^0.20.0 bisogna adattare in seguito al cambiamento di formato dei setting (annunciato qui https://developersitalia.slack.com/archives/CB6DCK274/p1539336857000100):

  • nell'array settings per inizializzare l'SP, la voce sp_singlelogoutservice è diventato un array per permettere la gestione di SLO multipli lato SP
  • tutte le chiamate ai metodi logout() e logoutPost() ora ricevono un primo parametro obbligatorio intero, che indica l'indice del sp_singlelogoutservice che si vuole usare:

qui un esempio: https://github.com/simevo/spid-php-lib-example/blob/master/index.php#L17

nel vs caso dovrebbe essere così:

diff --git a/app/config/config.yml b/app/config/config.yml
index 45c2ec1..caac402 100644
--- a/app/config/config.yml
+++ b/app/config/config.yml
@@ -52,7 +52,8 @@ spid_symfony:
     sp_entityid: 'http://localhost:8000'
     sp_key_file: '%kernel.root_dir%/../example/sp.key'
     sp_cert_file: '%kernel.root_dir%/../example/sp.crt'
-    sp_singlelogoutservice: 'http://localhost:8000/slo'
+    sp_singlelogoutservice:
+        - ['http://localhost:8000/slo', '']
     sp_org_name: 'dev-system'
     sp_attributeconsumingservice:
         - ["name", "familyName", "fiscalNumber", "email"]

ma mi viene difficile testare ...

@marcoalbarelli
Copy link
Collaborator

Se per te va bene lo chiudo io nei primi giorni della settimana prossima.
Casomai assegnamelo (da solo non riesco in questo repo)

@marcoalbarelli marcoalbarelli self-assigned this Oct 15, 2018
@marcoalbarelli
Copy link
Collaborator

L'aggiornamento è stato fatto
Ho rilasciato un po' di versioni del bundle, siamo arrivati alla v0.2.2-beta
Testo ancora un po' in staging sul progetto dove l'ho integrato e se va bene rilascio una v1.0.0
Fatto quello se trovo il tempo faccio una PR per aggiornare anche questo repo

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants