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

Failed parsing MLSD response on some servers #45

Open
luza opened this issue Nov 24, 2019 · 2 comments
Open

Failed parsing MLSD response on some servers #45

luza opened this issue Nov 24, 2019 · 2 comments

Comments

@luza
Copy link

luza commented Nov 24, 2019

Log fragment:

goftp: 0.110 #1 sending command MLSD /somedir
goftp: 0.144 #1 got 150-Using transfer connection
goftp: 0.144 error in ReadDir: failed parsing MLST entry: 

The server response was:

Type=dir;Size=0;Modify=20191124122657; subdir1<CRLF>
Type=dir;Size=0;Modify=20190808091946; subdir2<CRLF>
<CRLF>

Note the last empty line on which the parsing fails.
As to me, to fix this we could ignore empty lines in MLSD response. What do you think?

@tommed
Copy link

tommed commented Jan 29, 2021

I'm having the same problem. I'm assuming that because it's been ignored for 1yr, there's little point using this library.

@tommed
Copy link

tommed commented Jan 29, 2021

Fix: abc5130
Or just reference github.com/vipro-toolkit/goftp instead

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