Re: lynx-dev An ftp browsing bug

[Mike Castle]

On Fri, Jun 25, 1999 at 08:48:13PM -0500, Klaus Weide wrote:
> On Fri, 25 Jun 1999, Mike Castle wrote:
> > I did a simple hack to have lynx use NLST for NT, but it just gave me:
> > 
> >                                                                     /
> > directory
> > 
> > Current directory is /
> > 
> >      * text/plain
> 
> Hmm, I though Lynx was able to do something useful with just-filenames.

My bad.

Turns out that it was still parsing the output thinking it was the output
of LIST.

If I tell it to use NLST *and* move case WINDOWS_NT_SERVER: down by the
default: in parse_dir_entry(), it works as advertised.

It is even a simple fix until someone works out the details of possibly
using SITE DIRSTYLE.

--- HTFTP.c.orig        Tue May 25 12:13:02 1999
+++ HTFTP.c     Fri Jun 25 21:37:28 1999
@@ -931,7 +931,7 @@
 
        } else if (strncmp(response_text+4, "Windows_NT", 10) == 0) {
            server_type = WINDOWS_NT_SERVER;
-           use_list = TRUE;
+           /*use_list = TRUE;*/
            CTRACE(tfp, "HTFTP: Treating as Window_NT server.\n");
 
        } else if (strncmp(response_text+4, "MS Windows", 10) == 0) {
@@ -1908,7 +1908,6 @@
        case PETER_LEWIS_SERVER:
        case MACHTEN_SERVER:
        case MSDOS_SERVER:
-       case WINDOWS_NT_SERVER:
        case APPLESHARE_SERVER:
        case NETPRESENZ_SERVER:
            /*
@@ -2102,6 +2101,7 @@
            */
            break;
 
+       case WINDOWS_NT_SERVER:
        default:
            /*
            **  We can't tell if it is a directory since we only

One interesting thing that I saw netscape doing, however, was using size
and modtime.  That might be a way of getting more interesting information 
about something besides the current strategy of just giving up.

Example, if SIZE return nothing, then leave alone (don't know if SIZE is
broken or a directory), but if it did return something, we can fill in size
and assume a mime-type like we do for everything else.

Of course, that's a LOT more ftp traffic.  Not to mention, probably a bitch
to implement.

Oh, btw, I found another style of site that causes problems.  The mirror
package refers to it as a "dls" style.  From mirror.html:

  remote_fs=dls

00index.txt      189916  
0readme            5793  
1_x/                  = OS/2 1.x-specific files

   This is an ls variant used on some Un*x archives. It provides
   descriptions of known items in the listing. Set [174]flags_recursive
   to -dtR.

One site that does this is ftp://ftp.icsi.berkeley.edu.  (Of course, their
login message about using ls -lg instead doesn't work).

mrc
- 
       Mike Castle       Life is like a clock:  You can work constantly
  address@hidden  and be right all the time, or not work at all
www.netcom.com/~dalgoda/ and be right at least twice a day.  -- mrc
    We are all of us living in the shadow of Manhattan.  -- Watchmen