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.

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

bsains created an issue. See original summary.

DamienMcKenna’s picture

Version: 7.x-3.1 » 7.x-3.x-dev
Status: Active » Needs review

Don't forget to update the issue status to "needs review" when you upload a patch!

smurfxx’s picture

I 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.

bsains’s picture

I'd suggest that you check your permissions. This module simply instantiates the SSL connection.

smurfxx’s picture

Permissions 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?

bsains’s picture

Have you tested with just a plain old FTP connection?

couturier’s picture

Status: Needs review » Needs work

Sounds 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.

DamienMcKenna’s picture

Status: Needs work » Needs review

Status: Needs review » Needs work

The last submitted patch, backup_migrate-ftp-ssl-connect.patch, failed testing. View results

geek-merlin’s picture

Needs heavy reroll.

geek-merlin’s picture

Title: Enable FTPS support - SSL / TLS » Enable FTPS support
Priority: Normal » Major
Status: Needs work » Needs review
FileSize
3.44 KB

Rerolled 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.

Status: Needs review » Needs work

The last submitted patch, 11: backup_migrate-2663144-11-Enable-FTPS-support.patch, failed testing. View results
- codesniffer_fixes.patch Interdiff of automated coding standards fixes only.

DamienMcKenna’s picture

Status: Needs work » Needs review

Status: Needs review » Needs work
DamienMcKenna’s picture

Status: Needs work » Needs review

Status: Needs review » Needs work

The last submitted patch, 11: backup_migrate-2663144-11-Enable-FTPS-support.patch, failed testing. View results
- codesniffer_fixes.patch Interdiff of automated coding standards fixes only.

DamienMcKenna’s picture

The latest patch results in some errors (separate to the test infrastructure failing), so I think this needs some work.