Skip to content

Commit 0ea77a2

Browse files
fix(crl): derive missing CDP status from admin toggle in handler
When a certificate has no crlDistributionPoints extension, the handler now maps the status to DISABLED when external CRL validation is turned off, instead of always setting MISSING. This makes the source status align with the configured policy before signature validation runs. Signed-off-by: Vitor Mattos <1079143+vitormattos@users.noreply.github.com>
1 parent 0f20759 commit 0ea77a2

1 file changed

Lines changed: 4 additions & 1 deletion

File tree

lib/Handler/CertificateEngine/AEngineHandler.php

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -193,7 +193,10 @@ private function addCrlValidationInfo(array &$certData, string $certPem): void {
193193
$certData['crl_revoked_at'] = $crlDetails['revoked_at'];
194194
}
195195
} else {
196-
$certData['crl_validation'] = CrlValidationStatus::MISSING;
196+
$externalValidationEnabled = $this->appConfig->getValueBool(Application::APP_ID, 'crl_external_validation_enabled', true);
197+
$certData['crl_validation'] = $externalValidationEnabled
198+
? CrlValidationStatus::MISSING
199+
: CrlValidationStatus::DISABLED;
197200
$certData['crl_urls'] = [];
198201
}
199202
}

0 commit comments

Comments
 (0)