WordPress.org

Make WordPress Core

Changes between Initial Version and Version 1 of Ticket #34976, comment 59


Ignore:
Timestamp:
12/12/2015 05:02:25 PM (4 years ago)
Author:
dossy
Comment:

Legend:

Unmodified
Added
Removed
Modified
  • Ticket #34976, comment 59

    initial v1  
    44Can you list even one FTP server whose `NLST` verb does NOT return all files, thus requiring the "`-a`" option?  Can you even list one FTP server whose `NLST` verb even does anything when passed any option?
    55
    6 According to [https://tools.ietf.org/html/rfc1123#page-31 RFC1123, section 4.1.2.7], the `NLST` verb `MUST contain only a simple list of legal pathnames,` - dotfiles are legal pathnames, even if there's a semantic connotation that they are "hidden" by some programs by default.  Any FTP server whose `NLST` verb does not return all legal pathnames, which would include dotfiles, is breaking the specification set forth by the RFC.
     6According to [https://tools.ietf.org/html/rfc1123#page-31 RFC1123, section 4.1.2.7], the `NLST` verb's response `MUST contain only a simple list of legal pathnames,` - dotfiles are legal pathnames, even if there's a semantic connotation that they are "hidden" by some programs by default.  Any FTP server whose `NLST` verb does not return all legal pathnames, which would include dotfiles, is breaking the specification set forth by the RFC.
    77
    88Because of this, I suggest that the correct fix to this bug is to use `ftp_nlist()` and remove the "`-a`" option, as proposed by @ralbatross in [#comment:42 comment #42] above.