[PR #30675] fix: signin page stuck on loading when refresh token valid but access token expired #32921

Closed
opened 2026-02-21 20:52:19 -05:00 by yindo · 0 comments
Owner

Original Pull Request: https://github.com/langgenius/dify/pull/30675

State: closed
Merged: Yes


Summary

  • Fix signin page getting stuck on loading when user has valid refresh token but expired access token
  • Remove loginData?.logged_in from isLoading condition and use explicit isRedirecting state
  • Fix error handling in useIsLogin hook to treat any error as "not logged in"

Test plan

  • Verify signin page loads correctly when not logged in
  • Verify redirect works when user is already logged in
  • Verify page doesn't get stuck when access token expires but refresh token is valid
  • Verify error states (500, network errors) don't cause infinite loading

Fixes #30674

🤖 Generated with Claude Code

**Original Pull Request:** https://github.com/langgenius/dify/pull/30675 **State:** closed **Merged:** Yes --- ## Summary - Fix signin page getting stuck on loading when user has valid refresh token but expired access token - Remove `loginData?.logged_in` from `isLoading` condition and use explicit `isRedirecting` state - Fix error handling in `useIsLogin` hook to treat any error as "not logged in" ## Test plan - [ ] Verify signin page loads correctly when not logged in - [ ] Verify redirect works when user is already logged in - [ ] Verify page doesn't get stuck when access token expires but refresh token is valid - [ ] Verify error states (500, network errors) don't cause infinite loading Fixes #30674 🤖 Generated with [Claude Code](https://claude.com/claude-code)
yindo added the pull-request label 2026-02-21 20:52:19 -05:00
yindo closed this issue 2026-02-21 20:52:19 -05:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: langgenius/dify#32921