Skip to content

Commit 46bb8e2

Browse files
committed
rename to generateChallengeString
1 parent 469d260 commit 46bb8e2

File tree

2 files changed

+4
-4
lines changed

2 files changed

+4
-4
lines changed

Sources/WebAuthn/WebAuthnManager.swift

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -37,7 +37,7 @@ public struct WebAuthnManager {
3737
let userEntity = PublicKeyCredentialUserEntity(name: user.name, id: base64ID, displayName: user.displayName)
3838
let relyingParty = PublicKeyCredentialRpEntity(name: config.relyingPartyDisplayName, id: config.relyingPartyID)
3939

40-
let challenge = try generateChallenge()
40+
let challenge = try generateChallengeString()
4141

4242
let options = PublicKeyCredentialCreationOptions(
4343
challenge: challenge.base64EncodedString(),
@@ -241,7 +241,7 @@ extension WebAuthnManager {
241241
/// Generate a suitably random value to be used as an attestation or assertion challenge
242242
/// - Throws: An error if something went wrong while generating random byte
243243
/// - Returns: 32 bytes
244-
func generateChallenge() throws -> [UInt8] {
244+
func generateChallengeString() throws -> [UInt8] {
245245
var bytes = [UInt8](repeating: 0, count: 32)
246246
let status = SecRandomCopyBytes(kSecRandomDefault, bytes.count, &bytes)
247247
guard status == errSecSuccess else { throw WebAuthnManagerError.challengeGenerationFailed }

Tests/WebAuthnTests/HelpersTests.swift

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -4,8 +4,8 @@ import XCTest
44
final class HelpersTests: XCTestCase {
55
func testGenerateChallengeReturnsRandomBytes() throws {
66
let webAuthn = WebAuthnManager(config: .init(relyingPartyDisplayName: "123", relyingPartyID: "1", timeout: 60))
7-
let challenge1 = try webAuthn.generateChallenge()
8-
let challenge2 = try webAuthn.generateChallenge()
7+
let challenge1 = try webAuthn.generateChallengeString()
8+
let challenge2 = try webAuthn.generateChallengeString()
99

1010
XCTAssertNotEqual(challenge1, challenge2)
1111
}

0 commit comments

Comments
 (0)