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

OpenSSL 1.1.1c FIPS Causes Rewrite of Crypted Files #65

Closed
damaestro opened this issue Jul 16, 2019 · 2 comments
Closed

OpenSSL 1.1.1c FIPS Causes Rewrite of Crypted Files #65

damaestro opened this issue Jul 16, 2019 · 2 comments

Comments

@damaestro
Copy link
Contributor

It seems that when I clone an existing repo and use transcrypt on it with correct information I end up rewriting all crypted files with an upgraded openssl.

transcrypt master and 9a8a1f4 tested
openssl-1.1.1c-2.fc30.x86_64 "OpenSSL 1.1.1c FIPS 28 May 2019"

git clone proto://server/my.git
cd my
transcrypt -c aes-256-cbc -p 'the-existing-phrase'

All git operations cause a rewrite of the crypted files. For example:

[damaestro@earth my]$ git status
*** WARNING : deprecated key derivation used.
Using -iter or -pbkdf2 would be better.
[...] repeated for every file
On branch transcrypt-test
Changes not staged for commit:
  (use "git add <file>..." to update what will be committed)
  (use "git checkout -- <file>..." to discard changes in working directory)

	modified:   path/to/crypted.py
[...] all files listed
no changes added to commit (use "git add" and/or "git commit -a")

The files are locally readable.

@damaestro
Copy link
Contributor Author

Using v1.0.3 does not cause a rewrite and using master someone using an older version is still able to use the repo.

@elasticdog
Copy link
Owner

Hey @damaestro! There was a recent bug fix to ensure that salt generation was working consistently across operating systems, but it does require a re-encryption. This has been causing trouble for other people as well running in mixed version environments: Encrypted files appearing as changed in git. I hadn't predicted that a lot of people were running against the master branch, and it could cause further problems with new files, depending on who added them and with what version.

I was wanting to batch this change with some other potential breaking changes, but it's going to be a bit until I can circle back to those and make the changes cleanly. I'll publish a new release shortly so at least packages can be bumped and get people in a consistent place.

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

No branches or pull requests

2 participants