[PR #6645] 🍒[PM-33394] fix: Surface CookieRedirectException message during sync-on-unlock #12890

Closed
opened 2026-04-11 03:52:56 -05:00 by GiteaMirror · 0 comments
Owner

Original Pull Request: https://github.com/bitwarden/android/pull/6645

State: closed
Merged: Yes


🎟️ Tracking

PM-33394
Cherry-picked from #6643

📔 Objective

Surfaces the user-friendly error message from CookieRedirectException in the vault screen when a sync-on-unlock fails due to a cookie redirect, replacing the generic error message with actionable text.

  • Uses the Throwable.userFriendlyMessage extension (from the base branch) to extract the display-safe message in VaultViewModel.updateToErrorStateOrDialog
  • Falls back to the generic error message when no user-friendly message is available
  • Adds tests for both the no-data (ViewState.Error) and with-data (DialogState.Error) paths
**Original Pull Request:** https://github.com/bitwarden/android/pull/6645 **State:** closed **Merged:** Yes --- ## 🎟️ Tracking PM-33394 Cherry-picked from #6643 ## 📔 Objective Surfaces the user-friendly error message from `CookieRedirectException` in the vault screen when a sync-on-unlock fails due to a cookie redirect, replacing the generic error message with actionable text. - Uses the `Throwable.userFriendlyMessage` extension (from the base branch) to extract the display-safe message in `VaultViewModel.updateToErrorStateOrDialog` - Falls back to the generic error message when no user-friendly message is available - Adds tests for both the no-data (`ViewState.Error`) and with-data (`DialogState.Error`) paths
GiteaMirror added the pull-request label 2026-04-11 03:52:56 -05:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: github-starred/android#12890