Skip to content

Conversation

@joshtrichards
Copy link
Member

  • Resolves: #

Summary

Changes

  • Improved logic to ensure only files owned by the user are decrypted, preventing changes to shared or group data.
  • More robust error handling: file operation failures now raise exceptions and are properly logged.
  • Progress messages and logs are clearer and more informative.
  • Refactoring enhances code readability and maintainability.
  • Clean-up routines now reliably remove temp files after errors.
  • Return values explicitly distinguish between actions performed and files skipped.
  • Aligned the implementations of EncryptAll and DecryptAll more closely (still not perfect)

Benefits

Safer decryption for user data, enhanced reliability, and easier ongoing maintenance.

TODO

  • Update tests
  • Decide whether to do any of the inline TODOs while we're here
  • Double-check the UI output experience

Checklist

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.

2 participants