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

FR-17648 - Session management improvements and Native library upgrade #37

Merged
merged 1 commit into from
Sep 1, 2024

Conversation

frontegg-david
Copy link
Collaborator

Overview

This PR introduces significant improvements and critical bug fixes to the React Native SDK, focusing on token refresh mechanisms, OAuth error handling, and overall stability across both Android and iOS platforms.


Android

  • Token Refresh Improvements:

    • Implemented TimerTask for immediate token refresh in the foreground.
    • Utilized JobScheduler for efficient background token refresh.
    • Added a retry mechanism for failed background jobs.
    • Enhanced logic to seamlessly switch between foreground and background states.
    • Improved handling of edge cases, including near-expiry tokens.
    • Introduced token refresh status checks.
  • OAuth/Authorize Error Fix:

    • Resolved infinite loading issues on the /oauth/authorize page by correctly identifying and displaying backend errors.

iOS

  • Demo App and Documentation:

    • Fixed issues in the demo application and updated the README for clarity.
  • OAuth Error Handling:

    • Corrected error page handling for smoother OAuth authorization.
  • Codebase Cleanup:

    • Removed unnecessary logs and fixed a typo related to ephemeralSession.
  • Refresh Token Enhancements:

    • Improved token refresh management across app state transitions.

These updates enhance the SDK's reliability, performance, and user experience, making it more robust and efficient for both developers and users.

@frontegg-david frontegg-david merged commit 890fe5f into master Sep 1, 2024
2 checks passed
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