-
Notifications
You must be signed in to change notification settings - Fork 4
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
Endpoint per generazione metadata firmato #9
Conversation
- create le classi per l'elaborazione del metadata tramite XSD - aggiunte le classi per l'elaborazione della configurazione del SP - aggiunto middleware per esposizione metadata - integrati helper con funzioni relative a metadata - aggiunta funzione per serializzazione classi nel metadata - rimosse costanti non usate ed aggiunte altre necessarie - aggiunte funzioni per elaborazione elementi in XMLElement - aggiunta funzione per estrazione chiave pubblica da certificato - aggiornati namespace
- aggiunte delle estensioni mancanti dentro CieExtensions.cs - sistemato l'ordine dei tag esposti sotto l'elemento ContactPerson per evitare errori con spid-sp-test - sistemato CieOptions.cs - aggiornati gli XSD a seguito delle attività su italia/spid-sp-test#149 e su italia/spid-sp-test#148 - aggiornato README.md
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Implementare le action GitHub (come da repo SPID) per integrare i test di integrazione con spid-sp-test in fase di push, e modificare le versioni nel file di progetto da 1.1.3-prerelease4 a 1.2.0-prerelease (per verifica con pre-pubblicazione del pacchetto su nuget) e da 1.1.3 a 1.2.0
README.md
Outdated
E' possibile aggiungere nuovi ServiceProvider sia in maniera procedurale, in fase di `Startup`, come segue: | ||
|
||
```csharp | ||
.AddSpid(o => |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Fix naming
README.md
Outdated
sia utilizzando una classe che implementa l'interfaccia `IServiceProvidersFactory` e configurandola come segue: | ||
|
||
```csharp | ||
.AddSpid(o => |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Fix naming
README.md
Outdated
|
||
Infine, per poter esporre gli endpoint dei metadata relativi ai Service Provider registrati, sarà necessario aggiungere la seguente riga: | ||
```csharp | ||
app.AddSpidSPMetadataEndpoints(); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Fix naming
README.md
Outdated
app.AddSpidSPMetadataEndpoints(); | ||
``` | ||
|
||
Tutti i metadata generati vengono automaticamente esposti su endpoint diversi, che hanno come BasePath `/metadata-cie` (ad esempio, un metadata definito con NomeFile = `metadata.xml` verrà esposto sull'endpoint `/metadata-cie/metadata.xml`): il BasePath può essere cambiato, sovrascrivendo la proprietà `ServiceProvidersMetadataEndpointsBasePath` sulle SpidOptions nello `Startup.cs`. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Fix SpidOptions => CieOptions
aggiornate le github action con i test con spid-sp-test aggiunto il codice per generare i metadata relativi ai test di github action
Nooooo i refusi per mancata rinomina noooooooo.... 😂 PS. ho messo il tutto in draft in attesa del PR di spid-sp-test. Per ora ho provato tutto in locale e un test (relativo alla issue italia/spid-sp-test#121) da gli errori attesi. |
Ho provato a modificare le github actions aggiungendo questo:
perché nei log della build mi dava questo:
Ma poi ho trovato diversi link che dicono di aggiornare
non sono molto affine a python...cosa dici? |
Purtroppo su Python non so esserti molto d'aiuto. Prova a coinvolgere gli autori di spis-sp-test |
In realtà ho visto che la action fallisce perchè non trova il metadata: Exception: Metadata not found: server response with code 500 Ho visto che tenti di lanciare spid-sp-test con questi parametri: spid_sp_test --metadata-url https://localhost:5001/metadata-cie/metadata.xml --extra --debug ERROR --profile cie-sp-public Probabilmente la URL è sbagliata, prova con |
L'URL è corretta. I metadata generati dall'applicazione di esempio sono visibili a questi indirizzi: https://localhost:5001/metadata-cie/metadata.xml non so cosa fare onestamente...o provo a farmi una virtual machine ubuntu e testo la cosa anche lì oppure boh...tra l'altro ritorna un errore 500 quindi l'applicazione |
Puoi provare anche con WSL2, che ho usato personalmente per testare spid-sp-test su una macchina windows, senza dover scomodare necessariamente un host per vm ad hoc |
Grazie ci proverò. La cosa che non capisco è che se lancio Faccio ulteriori verifiche. |
Ho provato con WSL2 il quale inizialmente mi dava un problema 500 dovuto al fatto che non riuscisse a leggere correttamente il certificato (sembra per un problema di permessi che non riuscivo a confermare). Poi ho riavviato la mia macchina windows, causa aggiornamento e in parte a causa del fatto che la compilazione dell'applicativo continuava a darmi errori strani. Ora non ho più errori: a parte quelli di Ho controllato se ci fossero opzioni su Sto sperando che i problemi avuti fin ora siano collegati alle ultime problematiche avute dalla piattaforma https://www.githubstatus.com/history anche perché non so come loggare le cose per capire da dove deriva questo problema: continuo a cercare informazioni a riguardo. |
L'action fallisce perchè i due metadata impostati in Startup.cs vengono esposti con nome file metadata1.xml e metadata3.xml, ma la action cerca metadata.xml. Bisogna correggere da una delle due parti. Provvedo io alla modifica |
Fixes #8
Partendo dal codice di https://github.com/italia/spid-aspnetcore sono state aggiunte le seguenti componenti:
create le classi per l'elaborazione del metadata tramite XSD
aggiunte le classi per l'elaborazione della configurazione del SP
aggiunto middleware per esposizione metadata
integrati helper con funzioni relative a metadata
aggiunta funzione per serializzazione classi nel metadata
rimosse costanti non usate ed aggiunte altre necessarie
aggiunte funzioni per elaborazione elementi in XMLElement
aggiunta funzione per estrazione chiave pubblica da certificato
aggiornati namespace
aggiunte delle estensioni mancanti dentro CieExtensions.cs
sistemato l'ordine dei tag esposti sotto l'elemento ContactPerson per evitare errori con spid-sp-test
sistemato CieOptions.cs
aggiornati gli XSD a seguito delle attività su Updated CIE schema to avoid issue on tag order in ContactPerson spid-sp-test#149 e su fix: [CIE] xsd updated spid-sp-test#148
aggiornato README.md
Come evidenziato nel PR 149 di spid-sp-test rimane ancora da fixare una issue che impedisce di validate i metadata di un SP privato con solo il contatto amministrativo.