Skip to content

Handle connectivity issues in Recovery #1089

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

Open
wants to merge 1 commit into
base: development
Choose a base branch
from

Conversation

zaelgohary
Copy link
Contributor

@zaelgohary zaelgohary commented Jun 3, 2025

Changes

  • Connectivity Checking: Verifies internet connection before network operations.

  • Timeout Handling:

    • User info retrieval: 1-minute timeout.
    • PKid document retrieval: 1-minute timeout per operation.
    • Overall account recovery: 2-minute timeout.
  • User-Friendly Error Messages:

    • "No internet connection. Please check your network."
    • "Login timed out. Please check your network connection."
    • "Account recovery timed out. Please check your network connection."
  • Proper Error Logging: All network errors are logged with context for debugging.

  • UI Alignment: Error messages now properly align with form fields.

Related Issues

Tested Scenarios

  • Test with no internet connection
  • Test with slow/unstable network connections
  • Test timeout scenarios by simulating network delays
  • Verify error message alignment in UI
  • Test that existing functionality remains unchanged

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants