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

fix(async_tasks): add try catch to getChannelGroups async task #13673

Merged
merged 1 commit into from
Feb 22, 2024

Conversation

jrainville
Copy link
Member

Kind of fixes #13640

The issue itself was no reproducible even after dozens of restarts and following of the repro steps. It seems like this was an unfortunate DB close event that caused the app to close.

Adding a try catch won't actually fix that, since the DB is closed and also the communtiiesalready had a try/catch, but it will help for later.

Kind of fixes #13640

The issue itself was no reproducible even after dozens of restarts and following of the repro steps.
It seems like this was an unfortunate DB close event that caused the app to close.

Adding a try catch won't actually fix that, since the DB is closed and also the `communtiies`already had a try/catch, but it will help for later.
@status-im-auto
Copy link
Member

status-im-auto commented Feb 21, 2024

Jenkins Builds

Commit #️⃣ Finished (UTC) Duration Platform Result
✔️ 102c3ee #1 2024-02-21 20:47:26 ~6 min macos/aarch64 🍎dmg
✔️ 102c3ee #1 2024-02-21 20:47:40 ~6 min tests/nim 📄log
✔️ 102c3ee #1 2024-02-21 20:52:03 ~10 min tests/ui 📄log
✔️ 102c3ee #1 2024-02-21 20:53:54 ~12 min macos/x86_64 🍎dmg
✔️ 102c3ee #1 2024-02-21 20:57:52 ~16 min linux/x86_64 📦tgz
✔️ 102c3ee #1 2024-02-21 21:05:47 ~24 min windows/x86_64 💿exe

Copy link
Contributor

@igor-sirotin igor-sirotin left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

#10632 says hi 😄

@jrainville
Copy link
Member Author

#10632 says hi 😄

Yeah, I checked that issue yesterday to confirm that the channelGroup function was listed there.

@jrainville jrainville merged commit d6e13d7 into master Feb 22, 2024
8 checks passed
@jrainville jrainville deleted the fix/crash-get-channel-groups branch February 22, 2024 15:09
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.

App crash after updating from 2.27RC2 to 2.27RC2+
4 participants