In line with the new GDPR recommendations we should provide with some form of file encryption.

CommentFileSizeAuthor
#2 encryption-support-2975065-2.patch4.26 KBdinu r.

Comments

Alex Andrascu created an issue. See original summary.

dinu r.’s picture

dinu r. commented
StatusFileSize
new encryption-support-2975065-2.patch4.26 KB

Please check this patch. It uses defuse/php-encryption library.

https://github.com/defuse/php-encryption/blob/master/docs/classes/File.md

Alex Andrascu’s picture

Alex Andrascu commented

Can confirm this is working just fine. Struggled a bit with installing the `defuse/php-encryption` but after that all good. I suggest we should add some better helpers on how to install the library or even add it to the module composer.json.

  • Dinu Rodnitchi authored 6a1d9e9 on 8.x-4.x 
    Issue #2975065 by Dinu Rodnitchi, Alex Andrascu: Add encryption support
Alex Andrascu’s picture

Alex Andrascu commented
Status: Active » Reviewed & tested by the community

I've commited this to dev with some minor alterations. Good work Dinu! Can everyone please test and report ? Also the README file badly needs updating.

Status: Reviewed & tested by the community » Needs work

The last submitted patch, 2: encryption-support-2975065-2.patch, failed testing. View results

damienmckenna’s picture

damienmckenna
Location TN, USA
commented
Status: Needs work » Fixed

Seeing as it's already committed, lets do new issues if it needs further work.

Alex Andrascu’s picture

Alex Andrascu commented

Do you have any hints on why it failed the test Damien? That message doesn't tell me a lot.

Alex Andrascu’s picture

Alex Andrascu commented

Ah nevermind. Figured it out I think.

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.