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.
Thought it would be good to also enable connections via ftp_ssl_connect.
Adds a new option setting in FTP admin page to switch to the SSL connection method.
This option is only enabled is ftp_ssl_connect function exists and the openSSL module is enabled.
Comment | File | Size | Author |
---|---|---|---|
#11 | backup_migrate-2663144-11-Enable-FTPS-support.patch | 3.44 KB | geek-merlin |
Comments
Comment #2
DamienMcKennaDon't forget to update the issue status to "needs review" when you upload a patch!
Comment #3
smurfxx CreditAttribution: smurfxx commentedI tried this patch, backup&migrate connects to my SSL enabled ftp server but it can't write backups.
The errors are:
- FTP Error: Couldn't write to ./mybackup.mysql.gz when trying to save file on the ftp server.
- Could not run backup because the file could not be saved to the destination.
Comment #4
bsains CreditAttribution: bsains commentedI'd suggest that you check your permissions. This module simply instantiates the SSL connection.
Comment #5
smurfxx CreditAttribution: smurfxx commentedPermissions are ok, if I use any other ftp client from any other OS (pc, mac, android) using the same credentials it works and I can write into my folders. Is there any other log that can help me to understand why it can't write?
Comment #6
bsains CreditAttribution: bsains commentedHave you tested with just a plain old FTP connection?
Comment #7
couturier CreditAttribution: couturier as a volunteer commentedSounds like this patch either isn't working for everyone or needs more reviewers with adequate FTP connection. Also note that we have a new release since this was written, 7.x-3.2 on September 27, 2017.
Comment #8
DamienMcKennaComment #10
geek-merlinNeeds heavy reroll.
Comment #11
geek-merlinRerolled and cleaned up the code.
Setting major as we're in 2018 with a general consensus that unencrypted data rolling is a bug in itself.
Comment #13
DamienMcKennaComment #15
DamienMcKennaComment #17
DamienMcKennaThe latest patch results in some errors (separate to the test infrastructure failing), so I think this needs some work.