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

SendCommandAsync is gone from FtpClient? #12

Closed
AK107 opened this issue Jan 29, 2017 · 1 comment
Closed

SendCommandAsync is gone from FtpClient? #12

AK107 opened this issue Jan 29, 2017 · 1 comment
Assignees
Labels

Comments

@AK107
Copy link

AK107 commented Jan 29, 2017

Hi.

The last release drops subj, i.e. moves it to the FtpControlStream, but
But i need it to send custom commands, for example GetModificationTime (MDTM) or many other.

Can you share it?

p.s. one more question about:

"Microsoft.Extensions.Caching.Abstractions": "1.0.0",
"Microsoft.Extensions.Caching.Memory": "1.0.0",
"Microsoft.Extensions.Logging.Abstractions": "1.0.0"

this is depends from FULL NETStandard.Library

it is really necessary libraries? because in pure .net project (not net.standard) its pull all (20+) net.standard libs.

@sparkeh9
Copy link
Owner

Hi - this was omitted when refactoring. It has now been added back in @ v1.3.3 😄
Please see https://www.nuget.org/packages/CoreFtp/1.3.3

Regarding the listed dependencies - CoreFTP is aimed at the NETStandard API surface, so this is expected if being used in .NET Framework.

The caching is used in DNS lookup, and the logging is used throughout. If you can think of a way to stop lots of packages being included for .NET Framework while still providing the same level of functionality then by all means, issue a pull request.

NuGet is now a first class citizen and suddenly everything is now a NuGet package. The amount of packages required for inclusion in many libraries has gone up dramatically; While this is a bit overwhelming, I don't think this is necessarily a problem. In contrast to the monolithic .NET Framework, this "everything a package" makes apps much more lean at runtime.

@sparkeh9 sparkeh9 self-assigned this Jan 29, 2017
@sparkeh9 sparkeh9 added the bug label Jan 29, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants