dotnet/roslyn

Fix cascading diagnostics when using invalid crypto options with PublicSign

Aperta

#9198 aperta il 25 feb 2016

Vedi su GitHubGuida contributor
 (0 commenti) (0 reazioni) (0 assegnatari)C# (4257 fork)batch import
Area-CompilersBugConcept-Diagnostic ClarityFeature Requesthelp wanted

Metriche repository

Star
 (20.414 stelle)
Metriche merge PR
 (Metriche PR in attesa)

Descrizione

For instance, see PublicSignWithRelativeKeyPath in CompilationAPITests.vb.

There are two errors:

BC2014: the value 'test.snk' is invalid for option 'CryptoKeyFile'
+BC37254: Public sign was specified and requires a public key, but no public key was specified

In this case, only the first error is really necessary. This is true for almost any set of incompatible crypto options with public sign, including using CryptoKeyContainer or any set of mutually incompatible options.

See https://github.com/dotnet/roslyn/pull/9130 for the full set of diagnostics produced by incompatible options.

Guida contributor

Direzione di ricerca
Esamina la generazione di diagnostica per le opzioni crittografiche nel compilatore e assicurati che venga emesso solo il primo errore pertinente quando vengono fornite più opzioni incompatibili.
Tech stack
csharp
Dominio
compilers
Tipo issue
Bug
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-3 ore
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Nuova
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
C#
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
70