Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

LIST/NLST arguments #6

Closed
DanielAdolfsson opened this issue Feb 8, 2016 · 4 comments
Closed

LIST/NLST arguments #6

DanielAdolfsson opened this issue Feb 8, 2016 · 4 comments

Comments

@DanielAdolfsson
Copy link

According to RFC959, it should be possible to specify a directory when listing using NLST or LIST. It would seem they are both implemented in a way that would allow specifying masks (which has no backing the the RFC at all, as far as I can see), but ignores the path.

It is my understanding that FtpServer implements these commands incorrectly.

@fubar-coder
Copy link
Contributor

It's a bug if it ignores the path, but a ... flexible interpretation of the LIST/NLST arguments seems useful when I see what kind of FTP clients are out in the wild...

@DanielAdolfsson
Copy link
Author

I think it depends entirely on the command, though.

A more flexible interpretation of the LIST command is probably a great idea, as long as it adheres to the basic requirements of resolving paths.

However, Section 2.2.2 of RFC3659 (which is an extension to the earlier RFC) seem to explicitly forbid wildcarding the MLSD and MLST commands.

@DanielAdolfsson
Copy link
Author

You can ignore my previous comment, it would seem the M* commands are properly handled :)

@fubar-coder
Copy link
Contributor

Fixed in 1.3.1:

  • LIST now supports options and respects the given path
  • MLST returns data using the connection stream
  • LIST hides files and directories starting with a dot (except . and ..)
  • Several fixes regarding relative paths

LIST supports wildcards, but this is just a side effect, because LIST must support paths to file names.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants