italia / spid-aspnetcore

AspNetCore Remote Authenticator for SPID

Home Page:https://github.com/danielegiallonardo/spid-aspnetcore

Geek Repo:Geek Repo

Github PK Tool:Github PK Tool

Conformità ad "Avviso SPID n.19 versione 4" attributo "spid:entityType="spid:aggregator">"

marco-maroni-spot-software opened this issue · comments

Durante la cerficicazione ci è stato sollevata l'eccezione della mancata presenza di questo attributo del tag ContactPerson nel metadata generato come service provider:

image

Ma la libreria non lo genera o non vedo opzioni per generarlo.

Grazie

Per generare un metadata come aggregatore/aggregato, è necessario usare uno dei seguenti tipi:

  • ServiceProviderPublicFullAggregator
  • ServiceProviderPrivateFullAggregator
  • ServiceProviderPublicLightAggregator
  • ServiceProviderPrivateLightAggregator

I tipi come semplici ServiceProvider Public o Private non generano questi tag.

Grazie sì corretto mi era sfuggita quella parte di documentazione.

Impostando come tipo ServiceProviderPublicFullAggregator (che è il mio caso), non ho modo di impostare il FiscalCode che invece da specifiche sempra essere obbligatorio, tanto che il validatore online mi segnala come test non passato.

image

Nel tipo ServiceProviderPublicFullAggregator ci sono sia i campi AggregatorFiscalCode che AggregatedFiscalCode

Scusami mi sono sbagliato intendevo ServiceProviderPublicFullOperator ("pub-op-full")

image

Pubblicato package versione 1.3.12 contenente il fix

Il fix sembra funzionare ed ora il metadata generato passa tutte le validazioni anche strict/production.

Ora però sembra esserci un altro problema, provando con SPID demo con la nuova configurzione "ServiceProviderPublicFullOperator" mi dice che il metadata non è registrato anche se l'ho appena fatto.

Penso che il problema sia dovuto al fatto che il parametro EntityId nella configurazione io lo debba impostare senza https perché poi la librarie lo aggiunge quando compone l'entityid:

image

Quindi se per esempio impostao nel .config EntityId="localhost:5001" il metadata viene genarato correttamente con EntitiId="https://localhost:5001/pub-op-full/" ma poi questo questo url non corrisponde con quello che che viene passato in fase di signin saml e quindi SpidDemo dice che non trova il metadato perché non registrato.

Questo invece il modo in cui viene creato per il ServiceProviderPublic, qui l'https non viene messo. ed infatti in questo scenario il problema non si verifica.

image

Grazie

Si, è corretto, perchè nel caso degli aggregatori o degli operatori, l'entityId viene composto in automatico dalla libreria come composizione di 3 diversi parametri, mentre nel caso di SP pubblico o privato, lascio la scelta all'utilizzatore della libreria. Mi rendo conto che è un po' fuorviante, aggiornerò la documentazione. Nel frattempo comunque puoi tranquillamente utilizzare la versione 1.3.12 valorizzando opportunamente i campi nel caso di Operator (senza https)

Ok sì sono obbligato a fare una cosa del genere, non bellssimo ma sembra funzionare.
image

e poi
image