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

Cannot read directory error if filename contains ; char #55

Closed
crazy-max opened this issue Jul 18, 2019 · 3 comments
Closed

Cannot read directory error if filename contains ; char #55

crazy-max opened this issue Jul 18, 2019 · 3 comments

Comments

@crazy-max
Copy link
Owner

crazy-max commented Jul 18, 2019

If a file in a directory (FTP server) contains a ; the following error occurred :

Thu, 18 Jul 2019 19:32:18 UTC ERR Cannot read directory /test/ error="failed parsing MLST entry: type=file;size=7903718;modify=20190718185113;UNIX.mode=0644;UNIX.uid=1001;UNIX.gid=1001;unique=903g5b00215; file;test.txt" source=/test/

Bug related to secsy/goftp module.

@crazy-max crazy-max changed the title Cannot read directory if filename contains ; char Cannot read directory error if filename contains ; char Jul 18, 2019
crazy-max added a commit that referenced this issue Jul 20, 2019
@crazy-max
Copy link
Owner Author

crazy-max commented Jul 20, 2019

I have made a fork of secsy/goftp used by FTPGrab to fix some issues like this one and bring improvements :

  • Do not fail on incomplete MLSD / LIST entries
  • Fix MLST parser
  • Check if MLST supported otherwise fallback to LIST
  • Add CLNT and UTF8 feature support

@muirmanders If you are interested I can create PRs with those changes 👀

@muirmanders
Copy link

Yes, definitely interested in improvements. Some of your changes overlap with an old PR secsy/goftp#19.

Check if MLST supported otherwise fallback to LIST
goftp tries MLST without checking the feature because there are some servers that don't advertise the feature properly. I prefer the linked PR's approach of just "remembering" MLST doesn't work.

@crazy-max
Copy link
Owner Author

crazy-max commented Jul 20, 2019

Some of your changes overlap with an old PR secsy/goftp#19.

@muirmanders Yes indeed but this PR has too many features. I think I will create one PR per feature/fix.

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

No branches or pull requests

2 participants