Closed (fixed)
Project:
Field Encryption
Version:
7.x-1.x-dev
Component:
Code
Priority:
Major
Category:
Support request
Assigned:
Unassigned
Reporter:
Created:
9 Mar 2012 at 10:53 UTC
Updated:
15 Mar 2013 at 13:30 UTC
Jump to comment: Most recent file
Comments
Comment #1
tedbowMy plan is to change this module to use the Encryption module instead of having its own encryption funcitons.
Comment #2
caiovlp commentedI removed the encryption functions from this module and added the encrypt module as a dependency. I also took out the part where it was asking for a phrase as the encrypt module already handles keys in a more secure way.
Tested all three encryption methods (none, simple, mcrypt aes 256) that come out of the box when installing the encrypt module. Also tried changing between them and it works perfectly to my surprise.
I had to use utf8_encode/utf8_decode as encrypted strings were not utf8 and were blowing my database - I'm not sure if this is the best approach, but it works.
Let me know what you think.
Comment #3
wbridger commentedThanks caiovpl for the patch.
I tried #2 but got the error message below when applying the patch:
patch -p1 < using_encryption_module-1474930-2.patch
patching file field_encrypt.admin.inc
patching file field_encrypt.cache.inc
patching file field_encrypt.inc
patching file field_encrypt.info
Hunk #1 FAILED at 8.
1 out of 1 hunk FAILED -- saving rejects to file field_encrypt.info.rej
For info the content of field_encrypt.info.rej is
--- field_encrypt.info
+++ field_encrypt.info
@@ -8,4 +8,5 @@
files[] = field_encrypt.cache.inc
dependencies[] = field
-dependencies[] = field_ui+dependencies[] = field_ui
+dependencies[] = encrypt
For info, I had the module encrypt enabled when applying the patch.
I tried applying the patch prior and after enabling field_encrypt
Best.
Comment #4
caiovlp commented@wbridger thanks for trying! I think there's an issue with trailing whitespaces somewhere... I'm able to apply the patch on a Mac, but not on Windows. I tried regenerating the patch without luck. What worked for me was ignoring whitespaces:
git apply -v using_encryption_module-1474930-2.patch --ignore-whitespace
Checking patch field_encrypt.admin.inc...
Checking patch field_encrypt.cache.inc...
Checking patch field_encrypt.inc...
Checking patch field_encrypt.info...
Applied patch field_encrypt.admin.inc cleanly.
Applied patch field_encrypt.cache.inc cleanly.
Applied patch field_encrypt.inc cleanly.
Applied patch field_encrypt.info cleanly.
Can you please try that?
Thanks,
Caio
Comment #5
caiovlp commentedCommitted to 7.x-1.0-beta1.