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

DownloadDirectory may not work on custom systems due to internal FileListing #772

Closed
Adhara3 opened this issue Oct 30, 2021 · 4 comments
Closed

Comments

@Adhara3
Copy link
Contributor

Adhara3 commented Oct 30, 2021

FTP OS: Embedded

FTP Server: Custom

Computer OS: Windows

FluentFTP Version: 35.0.5

DownloadDirectory is internally performing a this.GetListing(remoteFolder, FtpListOption.Size | FtpListOption.Recursive) so using specific parameters that may not work as expected on all system.
Example: FtpListOption.NoPath may be needed in order to work.

This should be probably part of the FtpServerBase class.
Please note that the enum provides some behavioral flags (Recursive) while others are more archictural (NoPath, ForceList) and the BaseServer class should fill the latter kind

Regards
A

@robinrodricks
Copy link
Owner

Good idea, any suggestions how to implement this? A new property/method in FtpServerBase?

@Adhara3
Copy link
Contributor Author

Adhara3 commented Nov 23, 2021

Hi,

yes I would probably adda new method with a signature like the following
FtpListOption DecorateFileListOptions(FtpListOption options)

I can do the pull request if you want

Cheers
A

Adhara3 added a commit to Adhara3/FluentFTP that referenced this issue Nov 28, 2021
…ed flags

Issue robinrodricks#772, also calling that method in FtpClient.GetListing
@robinrodricks
Copy link
Owner

@Adhara3 please don't need to ask my permission to make pull requests, its hard to check so many active tickets. If you have any suggestion, please just file a PR and we can take it forward. I accept most PRs straightaway unless they cause a major conflict.

@robinrodricks
Copy link
Owner

Added to the bucket list. We will pick this up as and when we have free time. Comment on this issue if you want us to prioritize it. Thanks!

@robinrodricks robinrodricks closed this as not planned Won't fix, can't repro, duplicate, stale Sep 19, 2022
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