dotnet/roslyn

Fix cascading diagnostics when using invalid crypto options with PublicSign

Open

#9 198 ouverte le 25 févr. 2016

Voir sur GitHub
 (0 commentaires) (0 réactions) (0 assignés)C# (4 257 forks)batch import
Area-CompilersBugConcept-Diagnostic ClarityFeature Requesthelp wanted

Métriques du dépôt

Stars
 (20 414 stars)
Métriques de merge PR
 (Merge moyen 6j 17h) (256 PRs mergées en 30 j)

Description

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.

Guide contributeur

Direction de recherche
Examinez la génération de diagnostics pour les options cryptographiques dans le compilateur et assurez vous que seule la première erreur pertinente est émise lorsque plusieurs options incompatibles sont fournies.
Stack technique
csharp
Domaine
compilers
Type d'issue
Bug
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
2
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
1-3 heures
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Récente
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
C#
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
70