Added DownloadFile alias

[Redth]

I do a lot of downloading of external files in my build scripts.

This adds a new HttpAliases class with (so far) a single DownloadFile(string url, DirectoryPath outputPath) alias which does exactly what it looks like.

Eventually there could be more HTTP related aliases here such as DownloadString

[devlead]

As we're using this in a sync fashion we should wrap in a using block, so streams/responses gets deallocated.

[devlead]

@Redth hi looks good just left one comment.

[devlead]

@Redth Looks good, I'll take a look later to day and merge if I don't find anything else.

[RichiCoder1]

Random idea would also be an alias which does the same thing, but drops it in the appropriate "temp" directory and passed back a path/IFilePath. Fun food for thought.

[patriksvensson]

I like @RichiCoder1's idea as well. Ok if I add tomorrow?

[devlead]

@RichiCoder1 @patriksvensson download string / bytes would be nice additions to. if your at it;)

[patriksvensson]

Well, download a file as string/byte feels unsafe. Wouldn't it be better to
return a stream in that case and add convenience methods to convert that to
string/byte array? That way, everyone will opt in for that functionality
instead of doing it like that by mistake.
On Jul 3, 2015 16:24, "Mattias Karlsson" notifications@github.com wrote:

@RichiCoder1 https://github.com/RichiCoder1 @patriksvensson
https://github.com/patriksvensson download string / bytes would be nice
additions to. if your at it;)

—
Reply to this email directly or view it on GitHub
#273 (comment).

[devlead]

@patriksvensson methods allready exists on the standard .net WebClient which this pr uses
string DownloadString(String) and byte[] DownloadData(string address)
so would be easy to add aliases for.