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

TCPWare Wise Turtle Pull Request #6

wants to merge 41 commits into from


Copy link

Questa è la pull request relativa al Wise Turtle del gruppo TPCWare

antimo musone and others added 30 commits October 8, 2017 15:25
to take advantage of the previous changes (IdP classes and X409 class)
…ome Controller method SpidRequest(string idpLabel)

Remove commented code and extra LF in index.cshtml
…f SamlHelper class.

Add IdpSaml2Response DTO class.
Add SpidUserInfoHelper class
Added fields: statusCodeInnerValue, statusMessage and statusDetail.
Fix error when extracting metadata from unsuccessful response status.
If values are not availables, a "N/A" string is returned.
…rmCollection["SAMLResponse"] instead of collection[0]
Copy link

fradel commented Nov 13, 2017

buongiorno :)
Ho difficoltà a compilare il progetto, ricevo l'errore

TPCWare.Spid.Sdk.csproj : error : lo spazio dei nomi XML predefinito del progetto deve essere lo spazio dei nomi XML di MSBuild. Se il progetto viene modificato nel formato MSBuild 2003, aggiungere xmlns="" all'elemento . Se il progetto è stato modificato nel formato precedente 1.0 o 1.2, convertilo al formato MSBuild 2003. TPCWare.Spid.Sdk\TPCWare.Spid.Sdk.csproj

dove sbaglio? Grazie

Copy link
Collaborator Author

ncarandini commented Nov 14, 2017

@fradel con quale versione di Visual Studio lo stai aprendo?
La solution usa .NET Standard 2.0 e .csproj, quindi richiede VS 2017.3 (come minimo) e il .NET Core SDK.
Questi i link per scaricarli, se già non li hai:

Visual Studio 2017:
.NET Core SDK:

Copy link
Collaborator Author

Questa pull request è sostituita dalla più recente PR #7

@ncarandini ncarandini closed this Dec 10, 2017
@ncarandini ncarandini mentioned this pull request Dec 10, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
None yet
None yet

Successfully merging this pull request may close these issues.

None yet

2 participants