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

saml2 test work in local env #3

Merged
merged 5 commits into from Dec 20, 2017
Merged

saml2 test work in local env #3

merged 5 commits into from Dec 20, 2017

Conversation

@cicciodifranco
Copy link
Member

@cicciodifranco cicciodifranco commented Oct 8, 2017

La strategia per passport in versione base.

@alranel alranel reopened this Dec 6, 2017
@alranel
Copy link
Member

@alranel alranel commented Dec 6, 2017

Ciao @cicciodifranco, vedo che hai chiuso questa pull request. Forse è stato un errore, la riapro in modo che possiamo discuterne. A che punto è questo lavoro? Funziona? Cosa manca?

@cicciodifranco
Copy link
Member Author

@cicciodifranco cicciodifranco commented Dec 6, 2017

Ciao...
Allo stato attuale riesco ad effettuare il login e reperire tutti i dati... L'unica cosa è che l env di test non mi ritorna il session index e quindi non riesco a fare il logout.

@alranel alranel merged commit 5c7e48b into italia:master Dec 20, 2017
@alranel
Copy link
Member

@alranel alranel commented Dec 20, 2017

Thank you for your contribution, and welcome to the community of Developers Italia as maintainer of this repository, on which you now have full admin rights 😃
(Remember: with great power comes great responsibility.)

I also invited you to become part of the italia organization, so that you can display its logo on your profile (don't forget to change the visibility after you accept the invitation!).

@alranel
Copy link
Member

@alranel alranel commented Dec 20, 2017

@cicciodifranco riguardo al problema sul logout ti va di aprire una issue qui? https://github.com/italia/spid-testenv Così insieme ai maintainer del testenv e agli altri maintainer SPID vediamo di risolverlo

@cicciodifranco
Copy link
Member Author

@cicciodifranco cicciodifranco commented Dec 20, 2017

@lussoluca
Copy link
Member

@lussoluca lussoluca commented Dec 20, 2017

Ciao @cicciodifranco,
sto facendo delle prove per integrare questa libreria nel proxy dell'applicazione di Cittadinanza Digitale.

Usando l'idp di test (https://github.com/italia/spid-testenv) a me il session_index viene ritornato correttamente:

{ name_id: 'Luca', session_index: 'cf4cc4ef-eaf8-4b85-b6c6-a376fa9429e4', attributes: { placeOfBirth: [ 'Torino' ], gender: [ 'M' ], mobilePhone: [ '3334567890' ], idCard: [ 'librettoPensione KK121212 inps 2008-07-03 2018-07-03' ], familyName: [ 'Lusso' ], name: [ 'Luca' ], spidCode: [ '57753017-9feb-49c5-87f9-193ad4533ae0' ], dateOfBirth: [ '1979-02-24' ], countyOfBirth: [ 'IT' ], fiscalNumber: [ 'TMMEXQ60A10Y526X' ], email: [ 'llusso@wellnet.it' ] } }

però ho dovuto applicare delle modifiche al tuo codice (e alla libreria saml2-js):

nel caso di spid-pass ho aggiunto:

nameid_format: "urn:oasis:names:tc:SAML:2.0:nameid-format:transient", sign_get_request: true,

in spcfg e in saml2-js ho aggiunto:

'@AttributeConsumingServiceIndex': 1,

tra gli attributi di AuthnRequest (anche se secondo le regole tecniche di spid gli attributi AssertionConsumerServiceURL e ProtocolBinding dovrebbero essere sufficienti).

@cicciodifranco
Copy link
Member Author

@cicciodifranco cicciodifranco commented Dec 20, 2017

Ciao @lussoluca
grande!! allora provo questa configurazione e ti faccio sapere. Per la lib saml2-js non saprei... si potrebbe integrare nel repo.

@cicciodifranco
Copy link
Member Author

@cicciodifranco cicciodifranco commented Dec 20, 2017

Avevo configurato male il service provider nell' env di test: in pratica nn avevo abilitato "Enable Single Logout".

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

Successfully merging this pull request may close these issues.

None yet

3 participants
You can’t perform that action at this time.