Skip to content

Add cabf tls ecc key usage lint#1041

Merged
christopher-henderson merged 9 commits into
zmap:masterfrom
digirenpeter:add-cabf-tls-ecc-key-usage-lint
May 10, 2026
Merged

Add cabf tls ecc key usage lint#1041
christopher-henderson merged 9 commits into
zmap:masterfrom
digirenpeter:add-cabf-tls-ecc-key-usage-lint

Conversation

@digirenpeter

Copy link
Copy Markdown
Contributor

Adding a lint to check the following requirement

7.1.2.7.11 Subscriber Certificate Key Usage
The acceptable Key Usage values vary based on whether the Certificate’s
subjectPublicKeyInfo identifies an RSA public key or an ECC public key. CAs MUST ensure
the Key Usage is appropriate for the Certificate Public Key.

Table 56: Key Usage for ECC Public Keys

+-------------------+-----------+------------------+
| Key Usage         | Permitted | Required         |
+-------------------+-----------+------------------+
| digitalSignature  | Y         | MUST             |
| nonRepudiation    | N         | –                |
| keyEncipherment   | N         | –                |
| dataEncipherment  | N         | –                |
| keyAgreement      | Y         | NOT RECOMMENDED  |
| keyCertSign       | N         | –                |
| cRLSign           | N         | –                |
| encipherOnly      | N         | –                |
| decipherOnly      | N         | –                |
+-------------------+-----------+------------------+

There is a similar lint for RFC requirements (lint_ecdsa_allowed_ku) but that only checks the following requirement and not the CABF requirements.

// If the keyUsage extension is present in a certificate that indicates
// id-ecPublicKey in SubjectPublicKeyInfo, then the following values
// MUST NOT be present:
//
// keyEncipherment; and
// dataEncipherment.

@christopher-henderson christopher-henderson left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

As always, valuable and easy to read. Thank you!

@christopher-henderson christopher-henderson merged commit e9f6f87 into zmap:master May 10, 2026
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants