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 to prevent crashing when logging in with corrupted save file on Mac #270

Merged
merged 1 commit into from
Jan 17, 2025

Conversation

colezemind
Copy link
Contributor

@colezemind colezemind commented Jan 13, 2025

Added a Try Catch to AppleBridge to ensure the the editor doesn't crash if corrupted save data is passed along to be decrypted.
If save file is found but unable to be read, it is now just removed.

Docs Checklist

Please ensure you have addressed documentation updates if needed as part of this PR:

  • I have created a separate PR on the sequence docs repository for documentation updates: Link to docs PR
  • No documentation update is needed for this change.

…ing function call with Encryptor, this helps prevent crashing on MacOS
@colezemind colezemind requested review from a team as code owners January 13, 2025 19:52
@colezemind colezemind linked an issue Jan 13, 2025 that may be closed by this pull request
Copy link
Collaborator

@BellringerQuinn BellringerQuinn left a comment

Choose a reason for hiding this comment

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

Thank you for fixing this! Looks great 😄

Question for you: I noticed that our save file is called 'Cr.sav'. Is there any significance to that name? I'm wondering if maybe we should rename it to be something more specific like SequenceCredentials.sav? What do you think?

@BellringerQuinn BellringerQuinn merged commit 70dc3a1 into master Jan 17, 2025
1 of 2 checks passed
@BellringerQuinn BellringerQuinn deleted the 166-crsav-corruption-engine-crash branch January 17, 2025 15:31
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.

Cr.sav corruption engine crash
2 participants