search

italia / spid-testenv

Test environment for SPID (sample Identity Provider)
BSD 3-Clause "New" or "Revised" License
4 stars 0 forks source link

entityID incoerente #24

Closed alranel closed 6 years ago

alranel commented 6 years ago

Usando il testenv in locale, i metadati esposti dal backoffice riportano:

<md:EntityDescriptor ... entityID="idp.spid.gov.it"> 
...
<md:SingleLogoutService ... Location="https://idp.spid.gov.it:9443/samlsso" ResponseLocation="https://idp.spid.gov.it:9443/samlsso"/> 
<md:SingleLogoutService ... Location="https://idp.spid.gov.it:9443/samlsso" ResponseLocation="https://idp.spid.gov.it:9443/samlsso"/> 
<md:SingleSignOnService ... Location="https://idp.spid.gov.it:9443/samlsso"/> 
<md:SingleSignOnService ... Location="https://idp.spid.gov.it:9443/samlsso"/>

Vi sono alcuni problemi:

  1. Secondo le Regole Tecniche l'attributo entityID dovrebbe essere una URI ma non lo è.
  2. I tag <SingleLogoutService> e <SingleSignOnService>riportano un hostname non corrispondente con quello su cui gira il testenv (localhost o spid-testenv-identityserver).

Inoltre nelle asserzioni generate dal testenv si legge:

<saml2:Issuer Format="urn:oasis:names:tc:SAML:2.0:nameid-format:entity">spid-testenv-identityserver</saml2:Issuer>

Questa stringa è incoerente con i metadati di cui sopra e non è una URI come da Regole Tecniche.

mlocati commented 6 years ago

Vedere anche https://github.com/italia/spid-testenv-backoffice/pull/24 e https://github.com/italia/spid-testenv-identityserver/pull/13

mlocati commented 6 years ago

1 .Secondo le Regole Tecniche l'attributo entityID dovrebbe essere una URI ma non lo è.

Ho aggiornato italia/spid-testenv-backoffice#24 (vedi https://github.com/italia/spid-testenv-backoffice/pull/24/commits/dc4b7d3111511464a64568d254f6c41b93ba83cb ): ora l'entityID è un URL.

  1. I tag <SingleLogoutService> e <SingleSignOnService> riportano un hostname non corrispondente con quello su cui gira il testenv (localhost o spid-testenv-identityserver).

Sempre in italia/spid-testenv-backoffice#24 ho corretto tale discordanza.

damikael commented 6 years ago

Risolto con https://github.com/italia/spid-testenv-backoffice/commit/4454cf701c2ff3dbdf9ba640fee6eb64c394d311