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 601583 - After upgrade, Nautilus FTP adds space before some filenames
After upgrade, Nautilus FTP adds space before some filenames
Status: RESOLVED FIXED
Product: gvfs
Classification: Core
Component: ftp backend
1.4.x
Other Linux
: Normal major
: ---
Assigned To: gvfs-maint
gvfs-maint
Depends on:
Blocks:
 
 
Reported: 2009-11-11 17:38 UTC by jbooth
Modified: 2015-04-10 08:57 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
Invalid FTP shortcut (36.51 KB, image/png)
2010-04-02 04:08 UTC, Jean-Philippe Fleury
  Details
Window error that is displayed when we click on the shortcut (34.07 KB, image/png)
2010-04-02 04:09 UTC, Jean-Philippe Fleury
  Details
ftp: Fix directory parsing error on AIX. (1.99 KB, patch)
2015-02-28 13:02 UTC, Ross Lagerwall
committed Details | Review
ftp: Update license of ParseFTPList (2.72 KB, patch)
2015-02-28 13:02 UTC, Ross Lagerwall
committed Details | Review
ftp: Fix FTP date parsing of midnight for DOS format (1.10 KB, patch)
2015-02-28 13:02 UTC, Ross Lagerwall
committed Details | Review
ftp: Fix dir listing for windows long dates (1.08 KB, patch)
2015-02-28 13:02 UTC, Ross Lagerwall
committed Details | Review
ftp: Update license of ParseFTPList.h (2.81 KB, patch)
2015-03-17 12:48 UTC, Ondrej Holy
committed Details | Review

Description jbooth 2009-11-11 17:38:10 UTC
For months, I've been using Nautilus to browse an FTP connection to an AIX server. Since upgrading to Ubuntu 9.10 (karmic) last week, It's been padding a large percent of the file and directory names with a leading space.

When I try to open these directories, I get "Sorry, could not display all the contents of ' DIRNAME' The file is not a directory"

For files, in gedit, I get "Could not open the file ftp://...%20FILENAME. You do not have the permissions necessary to open the file."
Comment 1 André Klapper 2009-11-15 18:30:26 UTC
(By default it's prefered to file a bug against Ubuntu instead of GNOME first.)
Comment 3 Jean-Philippe Fleury 2010-04-02 04:08:22 UTC
Created attachment 157735 [details]
Invalid FTP shortcut

I think that I experience this bug too, but I don't have the "leading space" characteristic. Actually, since I updated to Gnome 2.29 (and now 2.30), when I access a FTP location, the link that is automatically added by Nautilus in the left panel is invalid. Here are the steps to reproduce the bug:

1) Open Nautilus.

2) Do "Ctrl+L" to access the location bar, write a valid FTP location (for example: ftp://user@server.ext) and press "Enter". The location is open and a shortcut is automatically added by Nautilus in the left panel. See the attached screen shot "ftp-shortcut.png".

3) Click on the shortcut previously added by Nautilus.

Actual result: a window error is displayed (see the attached screen shot "window-error.png"). The location isn't accessible by this shortcut, because the link is:

ftp://user@server.ext/%22/%22%20is%20the%20current%20directory

but it should be:

ftp://user@ftp.server.net/
Comment 4 Jean-Philippe Fleury 2010-04-02 04:09:08 UTC
Created attachment 157736 [details]
Window error that is displayed when we click on the shortcut
Comment 5 Cosimo Cecchi 2010-04-03 14:06:30 UTC
-> gvfs/ftp

This could be a bug in the FTP backend of GVfs, reassigning.
Comment 6 jbooth 2010-12-08 18:45:15 UTC
For anyone still looking into this, we've recently noticed this same problem with a few other FTP programs and discovered that on AIX, ls -l and ls through FTP show the date and time of a file if it is less than 6 months old. Older files get the time replaced with the year. e.g.:

drwxr-sr-x 5 nobody nogroup 256 Jun 30 10:51 bulletin
drwxr-sr-x 6 nobody nogroup 256 Nov 06 2009 cfi
drwxr-xr-x 6 nobody nogroup 256 Jan 15 2010 wpc

In this directory, Nautilus will only show the file with the time correctly. The other two have the problem with the leading space. It seems like somehow when Nautilus is parsing the ls output, that the extra space is coming from the end of the date columns somehow.
Comment 7 jbooth 2010-12-08 18:58:03 UTC
A co-worker of mine was looking into why other people in our office using Dreamweaver weren't having this problem and found this:

6. Specific FTP server issues

Macromedia has created a Dreamweaver MX 2004 extension for both Windows and Macintosh that resolves some FTP conflicts with IBM AIX and OS/400 servers. You should only install this extension if you are connecting to an IBM AIX or OS/400 server. Symptoms of the problem include:
Access Denied errors when attempting to access files or folders older than the current calendar year.
"Dreamweaver cannot determine the remote server time" errors and the Synchronize function not working.
Incorrect information in the Files panel, like extra spaces, junk characters, wrong kind of information in a particular field, etc. (Ref. 169317).
Comment 8 Ross Lagerwall 2015-02-28 13:02:17 UTC
Created attachment 298154 [details] [review]
ftp: Fix directory parsing error on AIX.

Fix from upstream:
http://hg.mozilla.org/mozilla-central/rev/b623418ab1de
https://bugzilla.mozilla.org/show_bug.cgi?id=543805
Comment 9 Ross Lagerwall 2015-02-28 13:02:22 UTC
Created attachment 298155 [details] [review]
ftp: Update license of ParseFTPList

Synced from upstream commit:
http://hg.mozilla.org/mozilla-central/rev/f4157e8c4107
https://bugzilla.mozilla.org/show_bug.cgi?id=716478
Comment 10 Ross Lagerwall 2015-02-28 13:02:27 UTC
Created attachment 298156 [details] [review]
ftp: Fix FTP date parsing of midnight for DOS format

Fix from upstream:
http://hg.mozilla.org/mozilla-central/rev/2c21b94a7775
https://bugzilla.mozilla.org/show_bug.cgi?id=961346
Comment 11 Ross Lagerwall 2015-02-28 13:02:34 UTC
Created attachment 298157 [details] [review]
ftp: Fix dir listing for windows long dates

Fix from upstream:
http://hg.mozilla.org/mozilla-central/rev/e10a2ca070e6
https://bugzilla.mozilla.org/show_bug.cgi?id=1061898
Comment 12 Ondrej Holy 2015-03-06 10:56:34 UTC
Thanks for looking at this! Please push them all, the parsing errors would be good to push also into stable...

It seems there isn't more relevant upstream patches to be applied - good job :-)
Comment 13 Ross Lagerwall 2015-03-08 09:13:50 UTC
Pushed to master and stable. Thanks for the review.
Comment 14 Ondrej Holy 2015-03-17 12:48:51 UTC
Created attachment 299589 [details] [review]
ftp: Update license of ParseFTPList.h

This is follow-up of commit 6a260ff, which updated license for ParseFTPList.c, but ParseFTPList.h should be updated also.
Comment 15 Ross Lagerwall 2015-03-17 12:55:31 UTC
Review of attachment 299589 [details] [review]:

Looks good.
Comment 16 Ondrej Holy 2015-04-10 08:57:17 UTC
Comment on attachment 299589 [details] [review]
ftp: Update license of ParseFTPList.h

commit f32b1a67eb08a02749604ecdb98bef8e7143a023