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.
Classic Graphics has a patch to add more helpful information to default.settings.php regarding connecting to MySQL by SSL.
Comment | File | Size | Author |
---|---|---|---|
#11 | guardr-settings-for-mysql-pdo-tls-1824946-11.patch | 2.7 KB | deekayen |
#5 | D7-ssl3.patch | 2.83 KB | shrop |
#2 | D7-ssl2.patch | 3.87 KB | shrop |
#1 | D7-ssl.patch | 4.55 KB | deekayen |
Comments
Comment #1
deekayen CreditAttribution: deekayen commentedHere's John Smith's patch.
Comment #2
shrop CreditAttribution: shrop commentedPatch without the "CG edit" comments. The original patch did work.
Comment #3
shrop CreditAttribution: shrop commentedDrush make builds work after applying the D7-ssl2.patch. Drupal install works okay from my testing. Closing this one for now as fixed.
Comment #4
deekayen CreditAttribution: deekayen commentedIt's an old patch. I think the pdo attributes were added in #1309278: Make PDO connection options configurable already for core. Just the settings.php stuff is what we'd want I think.
http://drupalcode.org/project/drupal.git/commitdiff/2f6d917af5b7367e2956...
Comment #5
shrop CreditAttribution: shrop commentedAttached is a new patch which only updates default.settings.php. Please confirm if this looks ok.
Comment #6
shrop CreditAttribution: shrop commentedOur sysadmin who worked on mysql over ssl for Drupal confirms that this patch looks right for D7. I will leave open for a bit to see if anyone has any issues with this or comments. Please test.
Comment #7
shrop CreditAttribution: shrop commentedPushed updated patch to stub file. http://drupalcode.org/project/guardr.git/blobdiff/ab2a5c46d6f4d7f47a3095...
Comment #8
deekayen CreditAttribution: deekayen commentedLooks fine to me.
Comment #9
shrop CreditAttribution: shrop commentedAdded an [additional comment in the stub file](http://drupalcode.org/project/guardr.git/commitdiff/e51a75b2709c32fbf4fe...) to point back to this issue. Marking fixed. Thanks!
Comment #10
deekayen CreditAttribution: deekayen commentedI noticed our servers at CG seem to have a different config then what the patch is documenting as a recommendation
Patch:
actual at CG
Comment #11
deekayen CreditAttribution: deekayen commentedComment #13
deekayen CreditAttribution: deekayen commented