Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Ongoing Verification Issues -- Android,iOS : ‘Verification Failed’ & Verification Error’ popup is shown while verifying the phone number. #3280

Closed
Lss-Ankit opened this issue Apr 2, 2020 · 39 comments · Fixed by #5656
Labels
Android Android specific issues bug Something isn't working Component: Attestations ios Priority: P1 Critical qa wallet

Comments

@Lss-Ankit
Copy link

Frequency: 100%
Android App version: Android Celo Alfa v1.6.2 (104794504)
Repro on Android Device: Samsung Galaxy J7 Prime (6.0.1)

Repro Steps:

  1. Launch the app and Perform sign up flow
  2. Navigate to verification flow
  3. Try to paste the code and Observe

Current Behavior: ‘Verification Error’ popup is shown while verifying verification code

Expected Behavior: ‘Verification Failed’ & ‘Verification Error’ popup should be not shown and account should be verified

Impact: User is unable to verify the account

Attachment: Android_Verification failed error message is shown.mp4 (Bug observed between 1:22 to 2:16 mins)

@Lss-Ankit Lss-Ankit added Android Android specific issues applications bug Something isn't working Priority: P0 Blocker qa labels Apr 2, 2020
@Lss-Ankit
Copy link
Author

@jeanregisser Verified with iOS test flight build : v1.6.2 (25). Verified on Device : iPhone 6+ (12.4.5) and observed that issue is also reproduce in iOS. 'Verification Failed' & 'Verification Error' popup is shown while verifying the phone number.

Attachment : iOS_Verification failed error message is shown

@Lss-Ankit Lss-Ankit added the ios label Apr 3, 2020
@Lss-Ankit Lss-Ankit changed the title Android : ‘Verification Failed’ & Verification Error’ popup is shown while verifying verification code. Android,iOS : ‘Verification Failed’ & Verification Error’ popup is shown while verifying verification phone number. Apr 3, 2020
@Lss-Ankit Lss-Ankit changed the title Android,iOS : ‘Verification Failed’ & Verification Error’ popup is shown while verifying verification phone number. Android,iOS : ‘Verification Failed’ & Verification Error’ popup is shown while verifying the phone number. Apr 3, 2020
@Lss-Ankit
Copy link
Author

@jmrossy Verified issue on Android build 1.7.0 (104894304) and observed that issue is still repro and also observed that user did not get initial escrow payments.

  • ‘Verification Failed’ & Verification Error’ popup is shown while verifying the phone number
    Attachment: Verification_Fail.mp4

@Lss-Ankit
Copy link
Author

Lss-Ankit commented Apr 27, 2020

@jmrossy Verified issue with latest Android build Android Celo Alfa build : v1.7.1 and observed that issue is still reproduced:

  • Android : ‘Verification Failed’ & Verification Error’ popup is shown while verifying the phone number

Attachment: Android_PhoneNumberVerification.mp4

@Lss-Ankit
Copy link
Author

@jmrossy Verified issue with latest iOS Test flight build : 1.7.1 (28) and observed that issue is still reproduced:

‘Verification Failed’ & Verification Error’ popup is shown while verifying the phone number

Attachment: iOS_Verification failed error message.mp4

@Lss-Ankit
Copy link
Author

@jmrossy Here are the iOS device logs. 'Verification failed' error is shown while verifying the phone number.

Attachment : iOS device log for verification failed.rtf

@jmrossy
Copy link
Contributor

jmrossy commented Apr 29, 2020

@Lss-Ankit Those are the general phone logs. In general, when we request logs, we mean the Celo App logs, which can be collected from the Settings page

@Lss-Ankit
Copy link
Author

@jmrossy 'Report' option is no longer available in android & iOS app. Verified on Android Celo Alfa build : v1.7.1 & iOS Test flight build : 1.7.1 (28). So, i am unable to take log from Settings page.
Attachment for
android : Android_Report option is not shown
Attachment for iOS : iOS_Report option is not shown

I have taken app log form device settings > analytics data. Here are the device log.
Device log: Celo_device_app log

@jmrossy
Copy link
Contributor

jmrossy commented Apr 30, 2020

@Lss-Ankit The 'Report' option is now called 'Support'

@Lss-Ankit
Copy link
Author

@jmrossy Thanks for the update. I have taken a log from support > contact us. Here the app log which having 'Verification failed' error while verifying the phone number
Attachment : iOS_Device log_Verification failed

@jmrossy
Copy link
Contributor

jmrossy commented May 1, 2020

@Lss-Ankit Thanks for sharing those logs.
It seems that verification failed because a transaction timed out. That's surprising because the limit of 40 seconds should be more than enough time for each transaction. Do you generally have any internet connectivity issues or is your connection solid? And does the app sometimes show the 'Connecting' banner?

@Lss-Ankit
Copy link
Author

@jmrossy Yes,i can say my connection is solid.
No ,i did not see 'Connecting' banner while performing testing.

@jmrossy
Copy link
Contributor

jmrossy commented May 4, 2020

@Lss-Ankit Thanks for clarifying! We are investigating a fix now.

@Lss-Ankit
Copy link
Author

Hi @annakaz and @jmrossy Today i have again verified issue using Android build Celo (Dev)_1.7.1 (101748676) and observed that issue is still reproduced.

Please find an attachmnet and logs for the same.
Attachment: VerificationFailed.mp4
Logs: celo_logs (1).txt

@annakaz
Copy link
Contributor

annakaz commented May 7, 2020

Note the logs from the comment above are now linked to issue #3747. The logs from this issue can be found at: https://celo-org.slack.com/files/UN7CEV7ED/F013059MA22/celo_logs_verificationfailed.txt?origin_team=T9SEZ6M9B&origin_channel=CNCUN1R0D

@Lss-Ankit
Copy link
Author

Hi @annakaz i have verified the issue with retrive data Saver and found that issue is still reproduced.

  • User is not able to verify the phone number even after user try with Retry with data saver button once verification is failed first time.
    Attachment: VerificationIssue.mp4

@Lss-Ankit
Copy link
Author

Hi @annakaz Please find the log for the above failure:

Attachment:
celo_logs_RetryWithDataSaver.txt

@Lss-Ankit
Copy link
Author

@nityas @jmrossy @annakaz i have verified this issue again on Android build 1.8.0 (104994308) and found the following:

@Lss-Ankit
Copy link
Author

Lss-Ankit commented May 27, 2020

@jmrossy Verified with Android play store (Internal build) v1.8.2 and observed that issue is still reproduced.

Attachment: Verificationfail.mp4
Logs: celo_logs (2).txt

@Lss-Ankit
Copy link
Author

Lss-Ankit commented Jul 30, 2020

Verified with latest IOS test flight Valora build v1.0.0(6) and observed that issue is reintroduced

  • ‘Verification Failed’ & Verification Error’ popup is shown while verifying the phone number.
  • Device Used: iPhone 6(12.4.7)
    VerficationErrorIssue

@Lss-Ankit
Copy link
Author

@annakaz Verified with Android Valora app build v1.0.0 (11) & IOS test flight build v1.0.0 (11) and found that issue is still reproduced.

  • ‘Verification Failed’ & Verification Error’ popup is shown while verifying the phone number.
    Note: i have try with retry with data saver, but still facing same error.
    Device:Google Pixel Xl (8.1) | iPhone 6+ (12.4.5)

@Lss-Ankit
Copy link
Author

@annakaz Verified with latest Android Valora app build v1.0.0 (12) but still not able to verify the phone number.

  • ‘Verification Failed’ & Verification Error’ popup is shown while verifying the phone number.
    Device:Google Pixel Xl (8.1)

@i1skn i1skn closed this as completed Oct 13, 2020
@Lss-Ankit
Copy link
Author

Hey @gnardini , @i1skn i have observed issue while perform testing using Android play store internal build v1.3.0(1004294316) & Test Flight build v1.3.0(27)

Issue observed for three times while verifying the phone number. Also, tried after enabling the data saver toggle but still facing same issue.
2 time we are able to verify phone number.

For now i am reopening this ticket.

For that i have taken logs which may help you.
Logs:
celo_logs.txt

@Lss-Ankit Lss-Ankit reopened this Oct 26, 2020
@mergify mergify bot closed this as completed in #5656 Nov 5, 2020
mergify bot pushed a commit that referenced this issue Nov 5, 2020
### Description

* Increase account unlock timeout to 10 minutes to fix referenced issue.
* Re-unlock account before verification start, so give at least 10 min guaranteed to a user to verify. It will also do it on resend messages.

### Other changes

Fix issue, when last confirmed attestation code is become empty after it is confirmed.

### Tested

Verification flow done

### Related issues

- Fixes #3280

### Backwards compatibility

Yes
@Lss-Ankit
Copy link
Author

@jeanregisser Verified task using Test flight build v1.4.0 (28) and Android play store (internal build) v1.4.0 (1004294317) and found that user is able to complete phone number verification flow with out facing any verification failed pop up.

Verified on : iPhone XR (13.2.2) and Google Pixel XL (8.1)

@Lss-Ankit
Copy link
Author

Hi @gnardini @nityas @jeanregisser We have verified issue with latest Test flight build v1.9.0 (41) and Android play store internal build 1.9.0 and observed that issue is reintroduced.

  • Verification Failed’ & Verification Error’ popup is shown while verifying the phone number.

Attachment

@Lss-Ankit
Copy link
Author

Hey @i1skn I have performed verification flow using latest Android Internal Build V1.9.2(1004294333) & Test Flight Build V1.9.2(43) & observed that we can able to perform verification flow. Also we have observed below observations.
For iOS:

  • User gets Verification error message when user try to perform verification while login with new account & from Confirm Now CTA.
    Note: We observed this issue when we performing whole process without any major time gap.

  • But able to perform verification without error message from setting.

Frequency: 40%

For Android:

  • User able to complete whole verification process in all three scenarios without any error.
    Verified on Devices: Samsung Galaxy A5(7.0), iPhone XR(13.1.3)

@i1skn
Copy link
Contributor

i1skn commented Jan 29, 2021

So, do I understand correctly that on iOS it sometimes failing only when you initiate verification from "Confirm Now" notification? Is possible to get logs when it does fail this way?

@tarikbellamine
Copy link
Contributor

I think this is related to the verification errors we discussed specific to 1.9.0, which we have fixed for the newest release candidate. I believe 1.9.2 has bot seen this issue reproduced. Is this correct @Lss-Ankit?

@Lss-Ankit
Copy link
Author

Hey @i1skn , @tarikbellamine I have verified this issue using latest Android Internal Build V1.9.2(1004294333) & Test Flight Build V1.9.2(44) & we have not observed any Verification fail in all scenario i.e. from While login with new account, From Confirm now CTA, Confirm Phone Number from setting & from Earn Rewards pills. In future if we observe any issue then we will update ticket for same.
Verified on Devices: iPhone SE(13.4.1), iPhone XR (13.1.3) , Samsung Galaxy A5(7.0)

@ValoraQA
Copy link

ValoraQA commented Mar 4, 2021

Hi @i1skn @tarikbellamine I have verified this issue using latest Android Internal Build V1.11.0(1004294337) & Test Flight Build V1.11.0(47) & observed that we are blocked for this issue.

Observations:

  • In latest build we have observed that user not able to get any attestation messages. For now we have blocked to test this this issue. In future we will update issue according to our observations.
    FYI: We have raised #65 issue for not receiving codes.
    Verified on Devices: Samsung Galaxy A5(7.0), iPhone SE (13.4.1)

@i1skn
Copy link
Contributor

i1skn commented Mar 4, 2021

Yeah, expect #65 to be resolved in the 1.11.1! Thanks for raising awareness to this!

@ValoraQA
Copy link

ValoraQA commented Mar 8, 2021

Hi @i1skn I have verified this issue using latest Android internal build V1.11.1(1004294338) & Test Flight Build V1.11.1(48) & observed following:
Observations:

  • For Android: It is observed that, user get Verification error popup while pasting last attestation code.

  • For iOS: It is observed that user get error while pasting first attestation code.

Hence reopening this ticket.

Frequency: 80%
Verified on devices: Samsung Galaxy A5(7.0), iPhone SE (13.4.1)
Attachment: VerificationErrorWhilePastingLastCode.mp4
Logs: Android_Logs.txt

@ValoraQA ValoraQA reopened this Mar 8, 2021
@i1skn
Copy link
Contributor

i1skn commented Mar 10, 2021

Hey @Celoqa, I'm not able to reproduce this bug on neither iOS neither Android devices. Also, I see that on the video there is an error happens even before the "enter code screen" appears, but in the description you've mentioned that the error appears on the "enter code screen". Android_Logs.txt does not provide much info as well. Can you please try to reproduce it and capture again the screen video as well as logs? Thanks!

@jeanregisser jeanregisser added Priority: P1 Critical and removed Priority: P0 Blocker labels Mar 10, 2021
@ValoraQA
Copy link

Hi @i1skn Here is detailed attachment using latest Android internal build V1.11.1(1004294338) & Test Flight Build V1.11.1(48).
Observations:

  • For Android: We have observed error while pasting last attestation code. This time frequency is 40%.
    Attachment: VerificationError.mp4
    Logs: Android_Logs.txt
  • For iOS: Now we have not observed any error while confirming phone number.
    Verified on devices: Samsung Galaxy A5(7.0), iPhone SE (13.4.1)

@jeanregisser
Copy link
Contributor

Should be fixed in 1.15.
Please reopen if it comes back ;)

@i1skn i1skn removed their assignment Jun 9, 2021
@ValoraQA
Copy link

Hi @jeanregisser We have verified this issue on latest builds Android internal V1.15.1 & Testflight build V1.15.1(64) and not observed any verifications issues.
Sure we will open this issue if comes back in future.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Android Android specific issues bug Something isn't working Component: Attestations ios Priority: P1 Critical qa wallet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

9 participants