Skip to content

Commit e991148

Browse files
committed
Review
1 parent 0aa8328 commit e991148

File tree

1 file changed

+17
-6
lines changed

1 file changed

+17
-6
lines changed

FirebaseAuth/Sources/Swift/Utilities/AuthRecaptchaVerifier.swift

Lines changed: 17 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -160,8 +160,12 @@
160160
actionString: String,
161161
fakeToken: String) async -> (token: String, error: Error?,
162162
linked: Bool, actionCreated: Bool) {
163-
if recaptchaClient != nil {
164-
return await retrieveToken(actionString: actionString, fakeToken: fakeToken)
163+
if let recaptchaClient {
164+
return await retrieveToken(
165+
actionString: actionString,
166+
fakeToken: fakeToken,
167+
recaptchaClient: recaptchaClient
168+
)
165169
}
166170

167171
if let recaptcha =
@@ -170,7 +174,11 @@
170174
// let client = try await recaptcha.fetchClient(withSiteKey: siteKey)
171175
let client = try await recaptcha.getClient(withSiteKey: siteKey)
172176
recaptchaClient = client
173-
return await retrieveToken(actionString: actionString, fakeToken: fakeToken)
177+
return await retrieveToken(
178+
actionString: actionString,
179+
fakeToken: fakeToken,
180+
recaptchaClient: client
181+
)
174182
} catch {
175183
return ("", error, true, true)
176184
}
@@ -181,12 +189,15 @@
181189
}
182190

183191
private func retrieveToken(actionString: String,
184-
fakeToken: String) async -> (token: String, error: Error?,
185-
linked: Bool, actionCreated: Bool) {
192+
fakeToken: String,
193+
recaptchaClient: RCARecaptchaClientProtocol) async -> (token: String,
194+
error: Error?,
195+
linked: Bool,
196+
actionCreated: Bool) {
186197
if let recaptchaAction =
187198
NSClassFromString("RecaptchaEnterprise.RCAAction") as? RCAActionProtocol.Type {
188199
let action = recaptchaAction.init(customAction: actionString)
189-
let token = try? await recaptchaClient!.execute(withAction: action)
200+
let token = try? await recaptchaClient.execute(withAction: action)
190201
return (token ?? "NO_RECAPTCHA", nil, true, true)
191202
} else {
192203
// RecaptchaEnterprise not linked.

0 commit comments

Comments
 (0)