Skip to content
This repository has been archived by the owner on Feb 19, 2019. It is now read-only.

[Enhancement] Added FTP support for the chocolatey file downloader. #137

Closed
wants to merge 3 commits into from

Conversation

austinsc
Copy link
Contributor

@austinsc austinsc commented Aug 9, 2012

Emulated existing Get-WebFile functionality using the FTP protocol. It could conceivably be merged together with the HTTP request, but the .net request/response objects are slightly different (i.e. the FTP request doesn't support cookies).

….ps1 script modeled after the existing Get-WebFile.ps1.
…-FtpFile.ps1 script modeled after the existing Get-WebFile.ps1."

This reverts commit 11e566e.
Added new Get-FtpFile.ps1 script modeled after the existing
Get-WebFile.ps1 and integrated the function into
Get-ChocolateyWebFile.ps1 so that URLs that begin with 'ftp' are
processed using Get-FtpFile.
@rismoney
Copy link
Contributor

did the line-feeds or spacing change? not sure why it shows the whole file changed...

@noomorph
Copy link

That's cool. And I can see it shows even the progress?
If so, that's really must-have.

@austinsc
Copy link
Contributor Author

Yup, it shows the progress as well. It is implemented almost identically to
the http version, I think I only left out the print to console
functionality (didnt see a need for it given the context).

On Sun, Aug 12, 2012 at 8:57 AM, noomorph notifications@github.com wrote:

That's cool. And I can see it shows even the progress?
If so, that's really must-have.


Reply to this email directly or view it on GitHubhttps://github.com//pull/137#issuecomment-7675430.

@ghost ghost assigned ferventcoder Aug 23, 2012
@ferventcoder
Copy link
Contributor

I have two pull requests for FTP support. Probably time to start a document on "So you want to contribute?"

@ferventcoder
Copy link
Contributor

I like this...quite a bit. Whatever you do though, don't update your repository with my latest changes. I think that document for contributions is going to get written soon.

We usually want people to contribute pull requests from a branch that is not master. That allows you to contribute multiple patches that way and doesn't get us into where we need to cherry pick your commits to keep the history in a nice straight line.

@austinsc
Copy link
Contributor Author

Any updates on this? I have a number of packages that i would like to upload pending this patch's acceptance. It is unclear to me if there is something else you would like me to do.

@ferventcoder
Copy link
Contributor

we'll be pulling it in soon.

@ferventcoder
Copy link
Contributor

This is in...thanks for the contribution!

@ferventcoder
Copy link
Contributor

This won't work of course until we release the next version of chocolatey (0.9.8.20), but that should be by the end of the month.

You can download and play with a prerelease version now.

@AnthonyMastrean
Copy link
Contributor

I'm sooo waiting for this feature, it was working by accident for a while. One of my earliest packages was on an FTP server (prime95). I think the enhancements around the CookieContainer stuff started breaking it. Anyway, this pull request looks good.

Something to consider in general, the HttpWebRequest and FtpWebRequest method Create is actually defined on WebRequest and can return any subtype. So, I can put "ftp://" in Get-WebFile and it breaks mysteriously.

@ferventcoder
Copy link
Contributor

This goes out into full release in a couple of days. So many new things being added. :D

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

Successfully merging this pull request may close these issues.

5 participants