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.

Sign up for GitHub

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

Jump to bottom

SessionSafeCoroutineWorker becomes stuck until a cache clear #6836

Closed
ouchadam opened this issue Aug 15, 2022 · 0 comments · Fixed by #6837
Closed

SessionSafeCoroutineWorker becomes stuck until a cache clear #6836

ouchadam opened this issue Aug 15, 2022 · 0 comments · Fixed by #6837
Labels
A-Notifications O-Uncommon Most users are unlikely to come across this or unexpected workflow S-Major Severely degrades major functionality or product features, with no satisfactory workaround T-Defect Something isn't working: bugs, crashes, hangs and other reported problems

Comments

@ouchadam
Copy link
Contributor

Steps to reproduce

  1. Pragmatically introduce an uncaught error in the doSafeWork within a child of SessionSafeCoroutineWorker
  2. Trigger the error
  3. Remove the error and reinstall the app
  4. Notice all future workers use the error flow instead of attempting to execute the work

Outcome

What did you expect?

The work to trigger

What happened instead?

It becomes stuck in the failure state until a cache clear (deleting the work manager database)

Your phone model

Emulator

Operating system version

Android 12

Application version and app store

1.4.32

Homeserver

No response

Will you send logs?

No

Are you willing to provide a PR?

Yes
@ouchadam ouchadam added T-Defect Something isn't working: bugs, crashes, hangs and other reported problems A-Notifications labels Aug 15, 2022
@ouchadam ouchadam mentioned this issue Aug 15, 2022
Merged
6 tasks
@ouchadam ouchadam added S-Major Severely degrades major functionality or product features, with no satisfactory workaround O-Uncommon Most users are unlikely to come across this or unexpected workflow labels Aug 18, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
A-Notifications O-Uncommon Most users are unlikely to come across this or unexpected workflow S-Major Severely degrades major functionality or product features, with no satisfactory workaround T-Defect Something isn't working: bugs, crashes, hangs and other reported problems
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Missing notifications due to stuck background workers element-hq/element-android
1 participant
@ouchadam