feat: Implement recaptcha verification during signup - #2360
Conversation
| private lateinit var forgotPasswordProgressDialog: Dialog | ||
| private lateinit var builder: AlertDialog.Builder | ||
| private var checkDialog: Boolean = false | ||
| private val RECAPTCHA_KEY = "6LcKhbMUAAAAAGFbYZeNFzqol-7EjOHUK5MvEeOE" |
The build is failing✨ Good work on this PR so far! ✨ Unfortunately, the Circle CI build is failing as of 9c4f090. Here's the output:
|
The build is failing✨ Good work on this PR so far! ✨ Unfortunately, the Circle CI build is failing as of 1048cc2. Here's the output:
|
|
@iamareebjamal please add the API key. The build is failing because it's not getting the key. |
mariobehling
left a comment
mariobehling
left a comment
There was a problem hiding this comment.
We cannot keep the API key in the app itself. This would be a security issue. It should be added during building of the app using Travis environment variables. Please implement it in that way.
|
Build should not fail if there is no API key The feature should only enable in presence of the API key |
|
I will update it accordingly. |
| SafetyNet.getClient(this).verifyWithRecaptcha(RECAPTCHA_KEY) | ||
| .addOnSuccessListener(this, OnSuccessListener { response -> | ||
| val userResponseToken = response.tokenResult | ||
| Log.d("KHANKI", "Started recaptcha verification") |
| val stringPassword = password.editText?.text.toString() | ||
| val stringConfirmPassword = confirmPassword.editText?.text.toString() | ||
| val stringURL = inputUrlSignUp.editText?.text.toString() | ||
| Log.d("KHANKI", "Signned up") |
4 participants
Fixes #2350
Changes:
Screenshots for the change:
