devlab/server-manager
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

fix: 401 logout loop — sync state update, prevent re-navigation #53

Merged
rafaga21 merged 1 commits from feat/push-monitoring-agent-v2 into master 2026-06-11 21:13:28 -04:00
Conversation 0 Commits 1 Files Changed 6 +14 -8
rafaga21 commented 2026-06-11 21:13:10 -04:00
Owner
Copy Link

Root cause: LoginViewModel.logout() was async (viewModelScope.launch),
so _uiState.isConnected remained true after navigating to login.
LoginScreen's LaunchedEffect immediately navigated back to dashboard,
where APIs failed with 401 again → infinite loop.

Fixes:

  • SessionManager: add _invalidating flag to prevent concurrent 401 handling
  • AuthInterceptor: check isSessionValid before invalidating
  • LoginViewModel: update _uiState synchronously in logout(), clear token
    before launching coroutine for prefs.clear()
  • LoginViewModel: call SessionManager.restoreSession() after successful login
  • MainActivity: remove restoreSession() from logout handler (was incorrectly
    re-validating session right after invalidation)

Bump to v1.7.1 (versionCode 10)

Root cause: LoginViewModel.logout() was async (viewModelScope.launch), so _uiState.isConnected remained true after navigating to login. LoginScreen's LaunchedEffect immediately navigated back to dashboard, where APIs failed with 401 again → infinite loop. Fixes: - SessionManager: add _invalidating flag to prevent concurrent 401 handling - AuthInterceptor: check isSessionValid before invalidating - LoginViewModel: update _uiState synchronously in logout(), clear token before launching coroutine for prefs.clear() - LoginViewModel: call SessionManager.restoreSession() after successful login - MainActivity: remove restoreSession() from logout handler (was incorrectly re-validating session right after invalidation) Bump to v1.7.1 (versionCode 10)
rafaga21 added 1 commit 2026-06-11 21:13:10 -04:00
fix: 401 logout loop — sync state update, prevent re-navigation 1885693f8d 
Root cause: LoginViewModel.logout() was async (viewModelScope.launch),
so _uiState.isConnected remained true after navigating to login.
LoginScreen's LaunchedEffect immediately navigated back to dashboard,
where APIs failed with 401 again → infinite loop.

Fixes:
- SessionManager: add _invalidating flag to prevent concurrent 401 handling
- AuthInterceptor: check isSessionValid before invalidating
- LoginViewModel: update _uiState synchronously in logout(), clear token
  before launching coroutine for prefs.clear()
- LoginViewModel: call SessionManager.restoreSession() after successful login
- MainActivity: remove restoreSession() from logout handler (was incorrectly
  re-validating session right after invalidation)

Bump to v1.7.1 (versionCode 10)
rafaga21 merged commit 3f95b061d3 into master 2026-06-11 21:13:28 -04:00
rafaga21 deleted branch feat/push-monitoring-agent-v2 2026-06-11 21:13:29 -04:00
Sign in to join this conversation.
Reviewers
No Reviewers
Labels
No items
No Label
Milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
agent rafaga21
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: devlab/server-manager#53
Delete Branch "feat/push-monitoring-agent-v2"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?