Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix AndFTP files download #474

Closed
wants to merge 1 commit into from
Closed

Fix AndFTP files download #474

wants to merge 1 commit into from

Conversation

Micdu70
Copy link
Contributor

@Micdu70 Micdu70 commented Nov 9, 2018

AndFTP does not seem to follow the (S)FTP directory path if written at the end of ftp(s):// or sftp:// links.
That's why I made this PR to add (S)FTP directory path to "remote_file" automatically.

@Micdu70 Micdu70 changed the title Add 'FTP(S) download directory' setting to fix AndFTP downloads Fix AndFTP files download Nov 10, 2018
@Micdu70
Copy link
Contributor Author

Micdu70 commented Nov 10, 2018

"Why make it simple when you can make it complicated?"
Fix has been updated.

@erickok
Copy link
Owner

erickok commented Nov 12, 2018

To be honest, AndFTP is hopelessy outdated (which I why I even wrote a little FTP Link Downloader app, which is minimal but at least functional).

Will probably merge, just have to check our the result.

@Micdu70
Copy link
Contributor Author

Micdu70 commented Nov 12, 2018

The only thing I would like is SSL session reuse functionality for FTPS (AndFTP doesn't have). But at least downloads are working for unsecure FTP and SFTP.

EDIT: Using rTorrent client, not tested for others.

@Micdu70
Copy link
Contributor Author

Micdu70 commented Nov 12, 2018

Just added forgotten SCP protocol which is compatible with AndFTP.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants