Skip to content

Commit 6a7730b

Browse files
committed
refactor: overriding #certificates enough to get the other methods return the correct thing
1 parent ac754db commit 6a7730b

1 file changed

Lines changed: 1 addition & 5 deletions

File tree

lib/webauthn/attestation_statement/android_safetynet.rb

Lines changed: 1 addition & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -16,10 +16,6 @@ def valid?(authenticator_data, client_data_hash)
1616
[attestation_type, attestation_trust_path]
1717
end
1818

19-
def attestation_certificate
20-
attestation_trust_path.first
21-
end
22-
2319
private
2420

2521
def valid_response?(authenticator_data, client_data_hash)
@@ -52,7 +48,7 @@ def attestation_type
5248
end
5349

5450
# SafetyNetAttestation returns full chain including root, WebAuthn expects only the x5c certificates
55-
def attestation_trust_path
51+
def certificates
5652
attestation_response.certificate_chain[0..-2]
5753
end
5854

0 commit comments

Comments
 (0)