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
update to defuse/php-encryption 2.1 #358
Conversation
a32c13d
to
d3d5b11
Compare
the db migration: as there's no way to encrypt using old version, should |
Sounds like that is the only option. I'm okay with it. |
i think will delay merge of this, there are too many changes in 3.4.0 release. have you yourself switched to code currently in master? i have tried mail sending, and have disabled several times due some issues. #353, #352, #350, #351 all somewhat related, those are solved now. but haven't tested in production download_emails.php part much either. |
I have not yet but I will next week. |
php-encryption/2.0.0:
NOTE: as the encrypted results are hex encoded by defuse/php-encryption, instead of base64, the result db columns may need to be increased. 1 byte encryption resulted 170 bytes encrypted result and 255 byte got exploded to 678 bytes.
so therefore not using defuse encoding, but keeping our own base64 implementation