-
Notifications
You must be signed in to change notification settings - Fork 34
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
MLST issues #36
Comments
Thanks for letting me know, and sorry for the late reply! I'm entering a new release cycle with uftpd and I'll have a look at it. |
Reproduced with tnftp in Linux Mint. Seems to be localized to MLST, MLSD works ... at least with ncftp. Will continue testing and see if I can get this fix into the v2.14 release, which I've planned for this coming weekend. |
Can't test on Ubuntu 18.04:
|
Source is available. I have no means to support all possible deb targets. |
i tried recent master and seems it fixed now, however another (new) issue appeared: each file is listed twice in LIST command. Server ran by
.. and another netcat connected just after PASV command responce:
Although command line ftp client shows only single file, likely it 'deduplicates' them internally. FileZilla for Windows however also shows duplicated entries in list. |
Too early assumed that MLST works, it works only for directories, but always fails for files. Here is log from netcat to uftpd:
Here is netcat session to pure-ftpd running on same machine:
|
Dammit, OK reopening 😞 Reproduced both issues, files listed twice, and mlst not working on files. Naturally the new one needs to go in its own bug report as a regression introduced in the latest release. I'll see what I can do over the next couple of days. Thank you all for testing and reporting back! |
Check that MLST works properly, issue #36 Signed-off-by: Joachim Wiberg <troglobit@gmail.com>
OK @elfmz, I've just pushed a bunch of changes to master which seem to fix the issues you reported. I'll probably do the v2.15 release later tonight, but I can postpone it if you'd like to help verify it first. Which I'd be very grateful for, of course! 😃 |
Didnt tested really throughfully, but current master looks now OK - both last issues not reproduced. |
Awesome, thank you for testing it out! <3 |
Release v2.15 is out now. Huge thanks to everyone for reporting, testing, and hanging in there! Merry Christmas 🎅 |
Looks like uftpd's MLST support is buggy. Details: elfmz/far2l#1067 (comment)
The text was updated successfully, but these errors were encountered: