secsy/goftp

Failed parsing MLSD response on some servers

Opened this issue · 2 comments

luza commented

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?

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

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