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

Verification attempts during initial sync on the SS proxy fail completely #1131

Closed
ara4n opened this issue Aug 23, 2023 · 1 comment · Fixed by #1138
Closed

Verification attempts during initial sync on the SS proxy fail completely #1131

ara4n opened this issue Aug 23, 2023 · 1 comment · Fixed by #1138
Assignees
Labels
A-Verification O-Occasional Affects or can be seen by some users regularly or most users rarely S-Major Severely degrades major functionality or product features, with no satisfactory workaround T-Defect Something isn't working: bugs, crashes, hangs and other reported problems

Comments

@ara4n
Copy link
Member

ara4n commented Aug 23, 2023

Steps to reproduce

Rick encountered:

  1. Log in on a big account for the very first time
  2. Sit spinning on sync while the SS proxy initial-syncs.
  3. Try to verify the new login (by initiating from EAX, i think?)
  4. Verif req goes through to another client (e.g. EW)
  5. EAX then hangs forever trying to complete the verif as the verif response is presumably stuck behind the initial sync.

Outcome

What did you expect?

Detect initial syncs from the proxy and warn the user explicitly and/or lock the app or show a beautiful animation during the very first initial sync?

Don't ask/let the user try to verify if they're still syncing, as we know it won't work.

Your phone model

No response

Operating system version

No response

Application version and app store

playstore 0.1.2 (40001020)

Homeserver

No response

Will you send logs?

Yes

Are you willing to provide a PR?

No

@ara4n ara4n added the T-Defect Something isn't working: bugs, crashes, hangs and other reported problems label Aug 23, 2023
@rxl881
Copy link

rxl881 commented Aug 23, 2023

Thanks for filing, yep, that's all accurate. I'll rageshake for logs if and when I get in.

@jmartinesp jmartinesp added S-Major Severely degrades major functionality or product features, with no satisfactory workaround O-Occasional Affects or can be seen by some users regularly or most users rarely A-Verification labels Aug 24, 2023
@jmartinesp jmartinesp self-assigned this Aug 24, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-Verification O-Occasional Affects or can be seen by some users regularly or most users rarely S-Major Severely degrades major functionality or product features, with no satisfactory workaround T-Defect Something isn't working: bugs, crashes, hangs and other reported problems
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants