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: Prevent passphrase regeneration when switching apps during encry… #13836

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

yuvraj-kumar-dev
Copy link

Here's a suggested description for your pull request regarding the encryption passphrase issue:


Pull Request Title: Fix Encryption Passphrase Handling

Description:

Issue Overview:
This pull request addresses the issue related to the encryption passphrase handling in our application. Previously, users experienced difficulties in securely managing their encryption keys, which could potentially lead to vulnerabilities.

Changes Made:

  • Implemented a more secure method for handling encryption passphrases.
  • Updated the encryption logic to ensure that passphrases are stored and transmitted securely.
  • Added validation to ensure the passphrase meets security requirements (length, complexity).
  • Improved error handling and user feedback during the passphrase entry process.
  • Updated relevant unit tests to cover the new passphrase handling logic.

Testing:

  • All unit tests have been updated and successfully pass.
  • Manual testing has been performed to ensure that the changes work as expected, particularly during encryption and decryption processes.

Impact:
This fix enhances the security of our application by ensuring that encryption passphrases are handled more securely, ultimately providing a better user experience.

…ption setup

Signed-off-by: Yuvraj Kumar <[email protected]>

Signed-off-by: Yuvraj Kumar <[email protected]>
:wq

git push --force-with-lease
@yuvraj-kumar-dev
Copy link
Author

Hi @tobiasKaminsky @alperozturk96 ,

I've addressed the issue with the passphrase regeneration during encryption setup. The changes are in the pull request. Could you please review and let me know if there are any concerns?

If everything looks good, could you merge it?

Thank you!

@alperozturk96
Copy link
Collaborator

@yuvraj-kumar-dev

Hi, thanks for the contribution 💯 I can only see changes in the test file.

Could you update the PR? A before-and-after demo would also be great, so we can reproduce the steps exactly.

Thanks 👍

Copy link

github-actions bot commented Nov 5, 2024

Hello there,
Thank you so much for taking the time and effort to create a pull request to our Nextcloud project.

We hope that the review process is going smooth and is helpful for you. We want to ensure your pull request is reviewed to your satisfaction. If you have a moment, our community management team would very much appreciate your feedback on your experience with this PR review process.

Your feedback is valuable to us as we continuously strive to improve our community developer experience. Please take a moment to complete our short survey by clicking on the following link: https://cloud.nextcloud.com/apps/forms/s/i9Ago4EQRZ7TWxjfmeEpPkf6

Thank you for contributing to Nextcloud and we hope to hear from you soon!

(If you believe you should not receive this message, you can add yourself to the blocklist.)

@tobiasKaminsky
Copy link
Member

Thank you for your contribution!
I fear there is something missing, as the commits only show changes to test classes…
Can you have a look?

@yuvraj-kumar-dev
Copy link
Author

@alperozturk96 @tobiasKaminsky Thank you for the feedback! I’ve double-checked the PR to ensure that all necessary implementation changes are included, especially in EncryptionUtils.java, as well as any relevant test updates. I’ve reviewed the commit history and confirmed that everything is up-to-date with the latest push.

If there’s anything specific you’d like me to verify or an additional file I may have missed, please let me know. Happy to make any further adjustments as needed. Thank you!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Encryption passphrase regenerates when you switch apps
4 participants