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

PRET support for downloads not working anymore. #7427

Closed
cyberduck opened this issue Sep 3, 2013 · 9 comments
Closed

PRET support for downloads not working anymore. #7427

cyberduck opened this issue Sep 3, 2013 · 9 comments
Assignees
Labels
bug fixed ftp version:4.3.1
Milestone

Comments

@cyberduck
Copy link
Collaborator

@cyberduck cyberduck commented Sep 3, 2013

17426b7 created the issue

PRET support for downloads not working anymore.

It seems to be the same defect as the #5772.

Login is working but download is not.
Error is :

 "500 You need to use a client supporting PRET (PRE Transfer) to use PASV." 

Tested and reproduced on Windows and OSX.

@cyberduck
Copy link
Collaborator Author

@cyberduck cyberduck commented Sep 3, 2013

@dkocher commented

Can you please post the transcript from the log drawer (⌘-L) of the Transfers window.

@cyberduck
Copy link
Collaborator Author

@cyberduck cyberduck commented Sep 3, 2013

@dkocher commented

Can you reproduce this issue with the latest snapshot build available.

@cyberduck
Copy link
Collaborator Author

@cyberduck cyberduck commented Sep 4, 2013

17426b7 commented

I've tested with the build 12675, same issue :

220 DRFTPD
AUTH TLS
234 AUTH TLS successful
SYST
530 Not logged in.
FEAT
530 Not logged in.
USER ****
331 Password required for ****.
PASS ********
230- 
230-         Welcome to the future of FXP:ing
230-      ___      _____________  ___
230-     / _ \____/ __/_  __/ _ \/ _ \
230-    /_ / __/ _/  / / / ___/ _ /
230-   /____/_/ /_/   /_/ /_/  /____/
230-                   DistRibuted FTP Daemon
230-                        http://drftpd.org
230 **** logged in successfully.
PBSZ 0
200 Command okay
PROT P
200 Command okay
TYPE I
200 Command okay
PASV
500 You need to use a client supporting PRET (PRE Transfer) to use PASV
PORT 192,168,1,22,213,66
501- ==YOU'RE BEHIND A NAT ROUTER==
501- Configure the firewall settings of your FTP client
501-   to use your real IP: **.**.**.**
501- And set up port forwarding in your router.
501- Or you can just use a PRET capable client, see
501    http://drftpd.org/ for PRET capable clients

@cyberduck
Copy link
Collaborator Author

@cyberduck cyberduck commented Sep 4, 2013

@dkocher commented

We fail to detect PRET support by the server because we receive a 530 reply when requesting features with FEAT before we are authenticated. We should retry the FEAT request after logging in.

@cyberduck
Copy link
Collaborator Author

@cyberduck cyberduck commented Sep 4, 2013

@dkocher commented

In c94145a.

@cyberduck
Copy link
Collaborator Author

@cyberduck cyberduck commented Oct 7, 2013

7c0bf6a commented

Lastest snashtop build

220 DRFTPD +
FEAT
530 Not logged in.
USER Admin
331 Password required for Admin.
PASS ********
230 Admin logged in successfully.
SYST
215 UNIX system type.
TYPE I
200 Command okay
PASV
500 You need to use a client supporting PRET (PRE Transfer) to use PASV
PORT 192,168,2,5,192,137
501- ==YOU'RE BEHIND A NAT ROUTER==
501- Configure the firewall settings of your FTP client
501-   to use your real IP: xxx.xxx.xxx.xxx
501- And set up port forwarding in your router.
501- Or you can just use a PRET capable client, see
501    http://drftpd.org/ for PRET capable clients

@cyberduck
Copy link
Collaborator Author

@cyberduck cyberduck commented Oct 8, 2013

@dkocher commented

The issue is that we cache the response to FEAT but DRFTPD returns a 530 error when requesting features before logged in.

@cyberduck
Copy link
Collaborator Author

@cyberduck cyberduck commented Oct 8, 2013

@dkocher commented

You can workaround this by connecting using a FTP-TLS connection as then we request server features only after successful login.

@cyberduck
Copy link
Collaborator Author

@cyberduck cyberduck commented Oct 8, 2013

@dkocher commented

In efbf042.

@iterate-ch iterate-ch locked as resolved and limited conversation to collaborators Nov 26, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug fixed ftp version:4.3.1
Projects
None yet
Development

No branches or pull requests

2 participants