Skip to content

Commit

Permalink
Add ineffective date to DSA lints. (#827)
Browse files Browse the repository at this point in the history
DSA is prohibited, so we can't maintain an up-to-date reference for how
a DSA key should be structured. Instead of checking prohibited DSA certs
against the old requirements, rely on lint_prohibit_dsa_usage.go
  • Loading branch information
dadrian committed Apr 6, 2024
1 parent f9496fa commit 2980c72
Show file tree
Hide file tree
Showing 3 changed files with 15 additions and 12 deletions.
11 changes: 6 additions & 5 deletions v3/lints/cabf_br/lint_dsa_correct_order_in_subgroup.go
Original file line number Diff line number Diff line change
Expand Up @@ -29,11 +29,12 @@ type dsaSubgroup struct{}
func init() {
lint.RegisterCertificateLint(&lint.CertificateLint{
LintMetadata: lint.LintMetadata{
Name: "e_dsa_correct_order_in_subgroup",
Description: "DSA: Public key value has the unique correct representation in the field, and that the key has the correct order in the subgroup",
Citation: "BRs v1.7.0: 6.1.6",
Source: lint.CABFBaselineRequirements,
EffectiveDate: util.CABEffectiveDate,
Name: "e_dsa_correct_order_in_subgroup",
Description: "DSA: Public key value has the unique correct representation in the field, and that the key has the correct order in the subgroup",
Citation: "BRs v1.7.0: 6.1.6",
Source: lint.CABFBaselineRequirements,
EffectiveDate: util.CABEffectiveDate,
IneffectiveDate: util.CABFBRs_1_7_1_Date,
},
Lint: NewDsaSubgroup,
})
Expand Down
5 changes: 3 additions & 2 deletions v3/lints/cabf_br/lint_dsa_shorter_than_2048_bits.go
Original file line number Diff line number Diff line change
Expand Up @@ -31,8 +31,9 @@ func init() {
Description: "DSA modulus size must be at least 2048 bits",
Citation: "BRs v1.7.0: 6.1.5",
// Refer to BRs: 6.1.5, taking the statement "Before 31 Dec 2010" literally
Source: lint.CABFBaselineRequirements,
EffectiveDate: util.ZeroDate,
Source: lint.CABFBaselineRequirements,
EffectiveDate: util.ZeroDate,
IneffectiveDate: util.CABFBRs_1_7_1_Date,
},
Lint: NewDsaTooShort,
})
Expand Down
11 changes: 6 additions & 5 deletions v3/lints/cabf_br/lint_dsa_unique_correct_representation.go
Original file line number Diff line number Diff line change
Expand Up @@ -29,11 +29,12 @@ type dsaUniqueCorrectRepresentation struct{}
func init() {
lint.RegisterCertificateLint(&lint.CertificateLint{
LintMetadata: lint.LintMetadata{
Name: "e_dsa_unique_correct_representation",
Description: "DSA: Public key value has the unique correct representation in the field, and that the key has the correct order in the subgroup",
Citation: "BRs v1.7.0: 6.1.6",
Source: lint.CABFBaselineRequirements,
EffectiveDate: util.CABEffectiveDate,
Name: "e_dsa_unique_correct_representation",
Description: "DSA: Public key value has the unique correct representation in the field, and that the key has the correct order in the subgroup",
Citation: "BRs v1.7.0: 6.1.6",
Source: lint.CABFBaselineRequirements,
EffectiveDate: util.CABEffectiveDate,
IneffectiveDate: util.CABFBRs_1_7_1_Date,
},
Lint: NewDsaUniqueCorrectRepresentation,
})
Expand Down

0 comments on commit 2980c72

Please sign in to comment.