spid-cie-php icon indicating copy to clipboard operation
spid-cie-php copied to clipboard

Errore durante la creazione del certificato spid-sp.crt

Open salvorapi opened this issue 3 years ago • 4 comments

Salve,

durante la fase di setup si presenta un problema nel caso in cui i parametri scritti nel file spid-php-openssl.cnf nella sezione dn sono vuoti.

I parametri richiesti in fase di setup sono:

  • spOrganizationName
  • spOrganizationDisplayName
  • spOrganizationIdentifier
  • spCountryName
  • spLocalityName

Bisognerebbe mettere un valore di default o rendere obbligatorio l'ingresso dei parametri.

salvorapi avatar Dec 21 '21 15:12 salvorapi

Trattandosi dei parametri di generazione del certificato, credo sia più appropriato rendere obbligatori i campi piuttosto che impostare valori di default.

eshiol avatar Aug 09 '22 20:08 eshiol

Trattandosi dei parametri di generazione del certificato, credo sia più appropriato rendere obbligatori i campi piuttosto che impostare valori di default.

L'obbligo dei campi è sicuramente una buona idea. Tuttavia, in fase di test, potrebbe essere utile avere dei valori di default per poter effettuare delle prove. Che ne pensi @heliosciancio ?

salvorapi avatar Aug 10 '22 06:08 salvorapi

Vada per il valore di default. Scelta in linea con quanto fatto per gli altri parametri.

Cosa usiamo: Località da specificare, Roma o altro?

eshiol avatar Aug 10 '22 08:08 eshiol

Alcuni valori di default sono già presenti: https://github.com/italia/spid-php/blob/303b5e01367d4db922dc731aeb4169e04a071df9/setup/Setup.php#L44

Dovrebbe mancare solo il codice ente: https://github.com/italia/spid-php/blob/303b5e01367d4db922dc731aeb4169e04a071df9/setup/Setup.php#L57 utilizziamo "code" anche se non dovrebbe essere un problema perchè spOrganizationIdentifier viene poi costruito insieme con spOrganizationCodeType quindi non dovrebbe essere mai vuoto

e la località: https://github.com/italia/spid-php/blob/303b5e01367d4db922dc731aeb4169e04a071df9/setup/Setup.php#L55 utilizziamo "locality".

damikael avatar Aug 10 '22 08:08 damikael