search

italia / spid-dotnet-sdk

SPID authentication library for .NET
BSD 3-Clause "New" or "Revised" License
28 stars 15 forks source link

#1 .NET Auth Plugin [WiP] #3

Closed ennioVisco closed 6 years ago

ennioVisco commented 7 years ago

Le interfacce principali della libreria sono implementate, manca tuttavia un po' di debug nell'interazione con il server.

fradel commented 6 years ago

Ciao, ho scaricato il progetto ma non riesco a compilarlo con vs2015, dopo aver ripristinato le reference riscontro un errore nella classe SPIDProvidersFactory,

private static Dictionary<string, (SPIDProtocols, AccountSettings)> providerSpecs = new Dictionary<string, (SPIDProtocols, AccountSettings)>()

non mi risulta esista un costruttore di Dictionary con questa sintassi, dove sbaglio?

ennioVisco commented 6 years ago

Pare che qualcuno abbia usato una nuova feature di .NET, le value tuples, qui dice:

Note that you will also need to install the Nuget package System.ValueTuple to use the new C# 7 value tuples features.

fradel commented 6 years ago

Libreria già installata, essendo inclusa nel package di nuget, ma continuo a ricevere l'errore Con il rischio di dire una sciocchezza, quella dichiarazione a me sembra errata

https://stackoverflow.com/questions/43764740/csetting-key-and-values-in-dictionary-with-tuple-of-strings

ennioVisco commented 6 years ago

Beh a noi compila, quindi direi proprio che non è errata :) Qui spiega meglio la differenza, come ti dicevo è una novità di C# 7.0 https://blogs.msdn.microsoft.com/mazhou/2017/05/26/c-7-series-part-1-value-tuples/

Prova magari specificando la versione del pacchetto System.ValueTuple 4.3.0

Ad ogni modo con VS2017 il problema non si verifica (credo sia tutto preinstallato nella versione opportuna).

alranel commented 6 years ago

Thank you for your contribution! Unfortunately, after an in-depth review we decided to merge another pull request. Your support is still very welcome and I'm sure @ncarandini will appreciate your help in improving this codebase.

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!).