Closed
Bug 105135
Opened 24 years ago
Closed 23 years ago
ftp sub-directory listing missing from html view of page
Categories
(Core Graveyard :: Networking: FTP, enhancement)
Tracking
(Not tracked)
VERIFIED
WONTFIX
Future
People
(Reporter: tever, Assigned: dougt)
References
()
Details
(Whiteboard: [PDT-])
Overview Description: a single ftp sub-directory listing is missing from the
html view in recent builds.
Steps to Reproduce:
1.) Go to url (internal only)
Actual Results: 2001-10-16-05-0.9.4/ directory not displayed
2001-10-15-10-trunk 10/15/01 11:57:00 am directory
2001-10-16-06-trunk 10/16/01 07:08:00 am directory
Expected Results:
there should be a directory listing for 2001-10-16-05-0.9.4/
Build Date & Platform Bug Found: winNT4, Linux rh6, mac osX 10/15 branch
Assignee | ||
Comment 1•24 years ago
|
||
WFM todays windows 2k build build (20011016)
Comment 2•24 years ago
|
||
Can you try clearing the cache?
Reporter | ||
Comment 3•24 years ago
|
||
clearing cache has no effect
Comment 4•24 years ago
|
||
I can't help unless I can reproduce this. dougt - can you repoduce this on the
branch?
Assignee | ||
Comment 5•24 years ago
|
||
see comment above. I can not reproduce this.
Reporter | ||
Comment 6•24 years ago
|
||
that is strange, I can reproduce it on different platforms and at least 4 other
people were commenting on seeing it in 'seamonkey-internal' today. There was a
link for downloading sent out because you couldn't go through the directory
listing. I wonder if the fact that both of you guys are working remotely today
has any significance. Doug, I didn't try win2k but I can show it to tomorrow if
you want.
Comment 7•24 years ago
|
||
Can someone who is seeing this give me a network trace (using ethereal), and the
results of running ./TestProtocols -verbose ftp://whatever ?
I don't have access to netscape-internal servers, so I can't fix this...
Assignee | ||
Comment 8•24 years ago
|
||
I see the problem.
201 lines that extend over the buffer boundary are disposed.
see nsIndexedToHTML::OnDataAvailable.
Bradley do you want to fix this, or should I? Any directory listing that has a
line on the boundary should illustrate this problem.
Tever, can you make us some test cases for this?
Comment 9•24 years ago
|
||
Does this happen on the trunk? Or is this branch only?
A quick glance though nsDirIndexParser didn't show anything - I got rid of that
old parsing code, and used the xul viewer's instead - see nsDirIndexParser on
the trunk
If this is branch only, and you want this fixed, then you should take this,
since I don't have a branch tree (my rewrite is not suitable for the branch.)
Assignee | ||
Comment 10•24 years ago
|
||
okay, it is mine.
pdt -
Any ftp directory listing view via html may have missing elements. The fix is
low risk (but not coded). Do we have time?
Assignee: bbaetz → dougt
Keywords: nsbranch
Assignee | ||
Comment 11•24 years ago
|
||
tever, can you try this in netscape 6.1?
Comment 12•24 years ago
|
||
tever: and, if you get time, confirm that this works on the trunk?
Reporter | ||
Comment 13•24 years ago
|
||
Doug, this does appear to have been present in 6.1. I can set up a test case
but I am not sure what you mean by '201 lines ...' at the moment. Can you clear
that up for me a bit.
Bradley, this does not appear to be a problem in the trunk, it works fine on
10/15 tr.
Comment 14•24 years ago
|
||
hmmm ... if this exact same bug is in 6.1, IMHO we should pass on this one.
Assignee | ||
Comment 15•24 years ago
|
||
Think of '201 lines' as a url that is listed in the html.
I agree with jamie. tever, please double check that 6.1 has the same behavior.
If it does, this bug is going to be resolved WONTFIX.
Assignee | ||
Comment 17•24 years ago
|
||
this should land after pdt is done with the branch for embedding.
Comment 18•24 years ago
|
||
I've verified that Mac OS X 6.1 build and Mac OS X 10-16 branch build both show
this problem.
Comment 19•24 years ago
|
||
PDT-, let's try and get it in later.
Keywords: topembed
Whiteboard: [PDT] → [PDT-]
Reporter | ||
Comment 20•24 years ago
|
||
Doug: double checked ... I still see the problem on a second machine with 6.1
(200110726)
Keywords: topembed
Whiteboard: [PDT-]
Updated•24 years ago
|
Whiteboard: [PDT-]
Updated•24 years ago
|
Comment 21•24 years ago
|
||
I have tested it on the last official beta of the embedded client and I don't
see s problem (brent, could you check and if you it works for you please remove
topembed from the bug).
Comment 22•24 years ago
|
||
I am unable to reproduce this using latest N6.2 with the following userAgent string:
Mozilla/5.0 (Windows;U;Win98;en-US;rv:0.9.4)Gecko/20011019 Netscape6/6.2
I can see and access the 2001-10-16-05-0.9.4
Assignee | ||
Comment 23•24 years ago
|
||
merak, the bug is there. The chance of hitting it is not great, but the bug is
there.
The fix is trival.
Comment 24•23 years ago
|
||
no patchie, no reviewie, no superreviewie, no takie. is there a r/sr patch for this?
Assignee | ||
Comment 25•23 years ago
|
||
crappe. :-)
I need to give this fixed for 0.9.6. Lets see if I have time.
Status: NEW → ASSIGNED
Target Milestone: --- → mozilla0.9.6
Assignee | ||
Comment 26•23 years ago
|
||
bmartin, because you can not see it, doesn't mean that it isn't there.
Severity: normal → enhancement
Target Milestone: mozilla0.9.6 → Future
Assignee | ||
Comment 27•23 years ago
|
||
This bug is only on the mozilla 0.9.4 branch. The bug is that directory
listings (LIST response) from the server which span a 4k internal buffer will be
truncated. The result is that the user viewed entry in HTML is undefined,
usually missing.
Marking WONTFIX. If anyone needs this for a customer, please reopen.
Status: ASSIGNED → RESOLVED
Closed: 23 years ago
Resolution: --- → WONTFIX
Comment 28•23 years ago
|
||
I didn't realize this was fixed on the trunk. what does the patch look like?
Comment 29•23 years ago
|
||
Valeski: It was fixed by my dirviewer rewrite, which removed teh html output
code, and made it use the generic parser. You don't want that patch for the
branch.
Updated•1 year ago
|
Product: Core → Core Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•