forked from mscdex/node-ftp
-
Notifications
You must be signed in to change notification settings - Fork 16
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
Expression unable to parse following directory listing #6
Comments
Soon I will look. |
icetee
added a commit
that referenced
this issue
Jun 16, 2018
@hajekj I changed the regexp. Do you have the opportunity to test? |
Works fine! Please release an updated version when possible. |
I've released v1.0.3 the new version. Thanks for submitting the issue. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
I have a following listing in FTP, the client doesn't seem to be able to parse it.
Update: It appears like the regex here is omitting
S
in the permissions and the directory listing appears to miss the size value - I wasn't able to find any RFC specification whether the size has to be specified.Update 2: I was able to find this: https://files.stairways.com/other/ftp-list-specs-info.txt and http://cr.yp.to/ftp/list/binls.html
Update 3: I found this library which seems to handle the parsing quite good: https://github.com/patrickjuchli/basic-ftp, maybe you can take some inspiration?
The text was updated successfully, but these errors were encountered: