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

Fixed a number of crashes currently affecting production #356

Merged

Conversation

mpretty-cyro
Copy link
Collaborator

  • Fixed a crash due to our ThreadSafe code using a struct instead of a class (rendering it non thread safe)
  • Fixed a crash which could occur on the home screen if the data loaded before the UI finished loading
  • (Hopefully) Fixed a crash which could occur when the OS optimised async execution to run immediately within an existing database transaction (potentially resulting in re-entrant database access)
  • Fixed an issue where the database read/write publishers weren't checking for a valid database state before actual query execution (only during the creation of the stream)

• Fixed a crash due to our ThreadSafe code using a struct instead of a class (rendering it non thread safe)
• Fixed a crash which could occur on the home screen if the data loaded before the UI finished loading
• (Hopefully) Fixed a crash which could occur when the OS optimised async execution to run immediately within an existing database transaction (potentially resulting in re-entrant database access)
• Fixed an issue where the database read/write publishers weren't checking for a valid database state before actual query execution (only during the creation of the stream)
@mpretty-cyro mpretty-cyro added the bug Something isn't working label Jan 14, 2025
@mpretty-cyro mpretty-cyro self-assigned this Jan 14, 2025
@stfsession stfsession merged commit 4113584 into session-foundation:dev Jan 14, 2025
2 checks passed
@mpretty-cyro mpretty-cyro deleted the fix/production-crashes branch January 14, 2025 02:06
@mpretty-cyro mpretty-cyro mentioned this pull request Jan 14, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants