Skip to content

Commit 61c6f9c

Browse files
committed
Fix incorrect approval of DSS
1 parent 0de337e commit 61c6f9c

2 files changed

Lines changed: 5 additions & 4 deletions

File tree

checks/tasks/tls/scans.py

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -74,7 +74,7 @@
7474
TLSExtendedMasterSecretEvaluation,
7575
)
7676
from checks.tasks.tls.tls_constants import (
77-
CERT_SIGALG_GOOD,
77+
CERT_SIGALG_SUFFICIENT,
7878
CERT_CURVES_GOOD,
7979
CERT_EC_CURVES_GOOD,
8080
CERT_EC_CURVES_PHASE_OUT,
@@ -389,7 +389,7 @@ def cert_checks(hostname: str, mode: ChecksMode, af_ip_pair=None, *args, **kwarg
389389
for cert in cert_deployment.received_certificate_chain:
390390
if not is_root_cert(cert):
391391
sigalg = cert.signature_algorithm_oid
392-
if sigalg not in CERT_SIGALG_GOOD:
392+
if sigalg not in CERT_SIGALG_SUFFICIENT:
393393
sigalg_bad[get_common_name(cert)] = sigalg._name
394394
sigalg_score = scoring.WEB_TLS_SIGNATURE_BAD
395395

checks/tasks/tls/tls_constants.py

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -6,14 +6,15 @@
66

77

88
# NCSC 3.3.2 / 3.3.5
9-
CERT_SIGALG_GOOD = [
9+
CERT_SIGALG_SUFFICIENT = [
1010
SignatureAlgorithmOID.RSA_WITH_SHA256,
1111
SignatureAlgorithmOID.RSA_WITH_SHA384,
1212
SignatureAlgorithmOID.RSA_WITH_SHA512,
1313
SignatureAlgorithmOID.ECDSA_WITH_SHA256,
1414
SignatureAlgorithmOID.ECDSA_WITH_SHA384,
1515
SignatureAlgorithmOID.ECDSA_WITH_SHA512,
16-
SignatureAlgorithmOID.DSA_WITH_SHA256,
16+
SignatureAlgorithmOID.ED25519,
17+
SignatureAlgorithmOID.ED448,
1718
]
1819

1920
# NCSC 3.3.2.1

0 commit comments

Comments
 (0)