Skip to content

Commit 59fc44f

Browse files
committed
Add log hint about policyTreeValidator setting
1 parent cde7099 commit 59fc44f

File tree

1 file changed

+8
-0
lines changed

1 file changed

+8
-0
lines changed

webauthn-server-core/src/main/java/com/yubico/webauthn/FinishRegistrationSteps.java

Lines changed: 8 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -30,6 +30,7 @@
3030
import COSE.CoseException;
3131
import com.upokecenter.cbor.CBORObject;
3232
import com.yubico.webauthn.attestation.AttestationTrustSource;
33+
import com.yubico.webauthn.attestation.AttestationTrustSource.TrustRootsResult;
3334
import com.yubico.webauthn.data.AttestationObject;
3435
import com.yubico.webauthn.data.AttestationType;
3536
import com.yubico.webauthn.data.AuthenticatorAttestationResponse;
@@ -52,6 +53,7 @@
5253
import java.security.cert.CertificateFactory;
5354
import java.security.cert.PKIXCertPathValidatorResult;
5455
import java.security.cert.PKIXParameters;
56+
import java.security.cert.PKIXReason;
5557
import java.security.cert.TrustAnchor;
5658
import java.security.cert.X509Certificate;
5759
import java.security.spec.InvalidKeySpecException;
@@ -565,6 +567,12 @@ public boolean attestationTrusted() {
565567
e.getIndex(),
566568
e.getMessage(),
567569
response.getResponse().getAttestationObject());
570+
if (PKIXReason.INVALID_POLICY.equals(e.getReason())) {
571+
log.info(
572+
"You may need to set the policyTreeValidator property on the {} returned by your {}.",
573+
TrustRootsResult.class.getSimpleName(),
574+
AttestationTrustSource.class.getSimpleName());
575+
}
568576
return false;
569577

570578
} catch (CertificateException e) {

0 commit comments

Comments
 (0)