After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 516704 - Be able to connect to an Active FTP Site
Be able to connect to an Active FTP Site
Status: RESOLVED FIXED
Product: gvfs
Classification: Core
Component: ftp backend
git master
Other All
: Normal enhancement
: ---
Assigned To: gvfs-maint
gvfs-maint
: 579851 (view as bug list)
Depends on:
Blocks:
 
 
Reported: 2008-02-15 17:01 UTC by Bill Iglesias
Modified: 2009-06-16 10:41 UTC
See Also:
GNOME target: ---
GNOME version: Unversioned Enhancement



Description Bill Iglesias 2008-02-15 17:01:36 UTC
I am not able to set up a Server using Active FTP from the Nautilus application.  It works fine with Passive FTP, but I could not find an option to tell it to use Active FTP.  Unfortunately, one of the sites which I administer does not allow Passive FTP.
Comment 1 Cosimo Cecchi 2008-03-22 12:55:05 UTC
Hi, FTP connection handling has been rewritten for 2.22.0. Is this still an issue with this latest version of Nautilus? Also, CC-ing Benjamin.
Comment 2 Benjamin Otte (Company) 2008-03-22 15:30:57 UTC
Yes, this is still not implemented.
Comment 3 Andreas Henriksson 2009-04-22 20:23:57 UTC
*** Bug 579851 has been marked as a duplicate of this bug. ***
Comment 4 Benjamin Otte (Company) 2009-06-11 09:33:13 UTC
I'm not sure we want this feature, as it:
- has various security implications
- probably requires quite a bit of code to implement (might be easier with gnio now?)
- isn't used very widely as almost all servers allow passive connections

Any reasons why it's a good idea to support it? One reason it would make sense would be server-to-server transfers (FXP), but that would require even more code inside gvfs.
Comment 5 Leandro Martínez 2009-06-11 13:09:47 UTC
Well, Benjamin, the only reason I've subscribed to this bug is because
I have to access a server with active ftp connectios and, of course,
I cannot change how the server is configured.

I don't understand the possible security implications this may be. But,
if they are important, then it would be at least nice to gvfs to say
something like:

The connection cannot be stablished because the server use
active ftp connections, which are unsecure. The best thing to
do is to warn the server administrator.

Currently, no error appears, a blank page is displayed, and the user
has no idea what is going on, just giving the impresion that 
gvfs has a bug.

In all cases, the best would be to allow the user to choose if he/she wants
to connect to such unsecure connection, and not to uncoditionally
forbid it, in which case the user will simply download another ftp
conection interface and connect to the server anyway.

Comment 6 Benjamin Otte (Company) 2009-06-15 21:10:13 UTC
commit 2839922c259b848d7689d245a055c628754dc116
Author: Benjamin Otte <otte@gnome.org>
Date:   Mon Jun 15 23:03:26 2009 +0200

    [FTP] Bug 516704 – Be able to connect to an Active FTP Site
    
    Add initial support for the PORT command. Support for EPRT and a
    non-ugly API are still missing.


It turns out the security issues are only for ftp servers (You don't want to allow some random client to tell you where to you open connections, that's a good method to do bad things to internal network), the code required was not that much and someone had to test the new gnio APIs for servers, so all in all quite a few reasons to implement this. ;)

I'll close this bug once EPRT support has landed.
Comment 7 Leandro Martínez 2009-06-15 21:15:27 UTC
Sorry Benjamin, I didn't understand your message. The bug is fixed, will not be fixed, there was some code changed? Someone (me?) can help testing something?
Comment 8 Benjamin Otte (Company) 2009-06-15 21:19:38 UTC
It means the first part of a fix has landed. If you don't use any non-standard things (like IPv6), git master should now connect to active ftp sites.
The code for IPv6 will follow later, and then I'll close the bug.
Comment 9 Benjamin Otte (Company) 2009-06-16 10:41:16 UTC
commit 19a6bf345fdb8d445e4c3683e4ca0af0a0031f0b
Author: Benjamin Otte <otte@gnome.org>
Date:   Tue Jun 16 11:16:27 2009 +0200

    [FTP] add EPRT support
    
    The code does not support some corner cases that are listed in the RFC
    (see inline comments), but I suspect those will never be hit. We can add
    those when they are hit.