Closed
Bug 28540
Opened 25 years ago
Closed 23 years ago
[RFE] FTP: nsFTPListingConv creates incorrect date in http-index-format
Categories
(Core :: Networking: FTP, enhancement, P3)
Tracking
()
VERIFIED
FIXED
Future
People
(Reporter: jonas.utterstrom, Assigned: bbaetz)
References
Details
(Whiteboard: needs review.)
Attachments
(1 file)
(deleted),
patch
|
Details | Diff | Splinter Review |
I have played around with "file" and "ftp" url directory listings. I noticed
that the ftp http-index-format was incorrectly formatted.
The code in "nsFTPListingConv.cpp" uses a nsIDateFormat object which depends on
the locale settings (sv_SE in my case).
http://lxr.mozilla.org/seamonkey/source/netwerk/streamconv/converters/nsFTPDirListingConv.cpp#877
This date should be formatted as a RFC822 date, which it is in the "file" url.
http://lxr.mozilla.org/seamonkey/source/netwerk/base/src/nsDirectoryIndexStream.cpp#226
So I took the code from the file url and modified nsFTPListingConv. See the
patch I have attached to this bug.
Reporter | ||
Comment 1•25 years ago
|
||
Reporter | ||
Comment 2•25 years ago
|
||
I have verified that the patch works on Windows NT also. And as a bonus, the ftp
directory view in Mozilla no longer thinks that files are directories in Linux.
The directory view in Windows worked before my patch, but it didn't follow the
date specification for http-index-format
(http://www.area.com/~roeber/file_format.html)
Comment 3•25 years ago
|
||
cc:ing Judson Valeski who wrote the original code.
Status: UNCONFIRMED → NEW
Ever confirmed: true
Reporter | ||
Comment 6•25 years ago
|
||
How can it be that this bug is still not fixed? Patches have a tendency to grow
old, so it is better to apply the patch earlier than later. I will also retract
that this bug doesn't exist in NT, it also appears when I changed locale from US
to Swedish.
Comment 7•25 years ago
|
||
I talked (mailed) with Judson Valeski about this. He sees the problem but does
not like the solution or thinks it should be done in another way. As I
understood him he thinks the http-index-format should be changed instead. With
the great beta push all people concentrated on PDT+ bugs and this one was not
marked as such.
I'm sure he will get back to this after the beta push.
Reporter | ||
Comment 8•25 years ago
|
||
That's fine by me. But I see no point in making a format locale-dependent when
it never is displayed for the user. It makes parsing simpler and faster to use
the RFC822 format. And it removes a dependency to the i18n/l10n modules.
Updated•25 years ago
|
Summary: FTP: nsFTPListingConv creates incorrect date in http-index-format → [RFE] FTP: nsFTPListingConv creates incorrect date in http-index-format
Target Milestone: M15 → M17
Comment 9•25 years ago
|
||
Moving post beta bugs to M18 which is now the post-beta milestone.
Target Milestone: M17 → M18
Updated•24 years ago
|
Updated•24 years ago
|
Target Milestone: mozilla0.9.1 → Future
Comment 12•23 years ago
|
||
to bradley.
Assignee: dougt → bbaetz
Component: Networking → Networking: FTP
Assignee | ||
Comment 13•23 years ago
|
||
This is fixed by my dirviewer rewrite, although according to bug 38014 it
doesn't work on windows. helpwanted in tracking that down.
Status: NEW → RESOLVED
Closed: 23 years ago
Resolution: --- → FIXED
Comment 14•23 years ago
|
||
+makingtest:
I'm not very saavy about this issue, so if there are an specific examples
someone can provide, or suggested testcases, that would be great. I'll work on
reading as many relevant bugs as I can.
Comment 15•23 years ago
|
||
Jonas: if this works the way you expect, can you mark your bug "VERIFIED"?
Keywords: verifyme
Comment 16•22 years ago
|
||
VERIFIED, this bug is subsummed by 38014. If there is a problem w/ dates, file a
new bug.
Status: RESOLVED → VERIFIED
Comment 17•22 years ago
|
||
[RFE] is deprecated in favor of severity: enhancement. They have the same meaning.
Severity: normal → enhancement
You need to log in
before you can comment on or make changes to this bug.
Description
•