Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
As it stands now, the module needs the key file to be writable at all times, even if the key file already exists and has a key inside. This patch allows administrators to set permissions on the key file to be read-only, to protect against accidental (or malicious) overwrites. Patch attached.
Comment | File | Size | Author |
---|---|---|---|
#7 | encrypt.install.allow-non-writeable-path.patch | 940 bytes | davidwhthomas |
encrypt.install.patch | 970 bytes | Nathan Goulding | |
Comments
Comment #2
Nathan Goulding CreditAttribution: Nathan Goulding commentedNot sure why this didn't pass. Can someone take a look?
Comment #3
Nathan Goulding CreditAttribution: Nathan Goulding commentedComment #4
Nathan Goulding CreditAttribution: Nathan Goulding commentedencrypt.install.patch queued for re-testing.
Comment #6
greggles@Nathan Goulding - see the "view details" link "SummaryFAILED: [[SimpleTest]]: [MySQL] Unable to apply patch encrypt.install.patch. See the log in the details link for more information."
From reading the log it looks like it could be an issue with the way the branches are setup.
Comment #7
davidwhthomas CreditAttribution: davidwhthomas commentedHere's an adjusted version of the patch for D7
Comment #8
gregglesFor what it's worth: I believe this feature already works properly in the 7.x-2.x branch.
Comment #9
gisleDrupal 6 is depreciated.