makomweb / emkay.s3 Goto Github PK
View Code? Open in Web Editor NEWEmkay S3 folder publisher can be used for publishing recursively the folder content to your S3 bucket.
Emkay S3 folder publisher can be used for publishing recursively the folder content to your S3 bucket.
There's a ton of extra junk in here that seems unnecessary to me. First of all, Interfaces that all only have one implementation is a bit useless. Is there a concrete case where other instances are used?
Moq is used in unit tests, but Moq can be used to mock concrete classes.. so having all the extra interfaces just to support Moq (which doesn't need them) seems overkill to me. I realize this can be a contentious issue as some people don't like to mock concrete classes, but it adds lots of extra code and layers to deal with with absolutely zero benefit.
The other one that I find extraneous along the same lines is the S3ClientFactory. There's only one S3Client instance.. So Factory is unnecessary here. Driving it home, the implementation is one line!!
public class S3ClientFactory : IS3ClientFactory
{
public IS3Client Create(string key, string secret)
{
return new S3Client(key, secret);
}
}
}
Before I do a PR, is the project maintainer open to these changes?
Hey,
It may be me but I think your repo is now broken:
I am going to roll back to the previous version. I stopped after getting error 3. I hope this helps you.
Adam
Hey guys,
Any chance you can update the nuget package if it now works with the new amazon sdk?
Thank you for Emkay.S3.
I would like to request the ability to specify the buffer size when uploading files to S3.
For example,
<PublishFiles
Key="$(Key)"
Secret="$(Secret)"
Bucket="$(Bucket)"
SourceFiles="@(SourceFiles)"
DestinationFolder="$(DestinationFolder)"
BufferSizeKiloBytes="65536"
TimeoutMilliseconds="$(TimeoutMilliseconds)" />
It seems the default buffer size is 8k but if it can be set to a higher value from MSBuild, uploading large files will be faster.
Thank you!
When an error occurs during publishing, only the message of the exception is logged:
https://github.com/makomweb/Emkay.S3/blob/master/Emkay.S3/PublishFiles.cs#L50
I would like to be able to see the entire details of the exception, including the stack trace to help diagnose and troubleshoot the issue.
I am currently experiencing an error and this is the only error message in the build log:
Publishing folder has failed because of A WebException with status RequestCanceled was thrown.
I would probably make a pull request to the forked repository here:
https://github.com/Dealermade/DM.Emkay.S3
Thank you.
I was looking to add a couple things, but the code seems like it could use some cleanup.
One of these things is separate method for "WithHeaders". Why bother? Why not just call PublishFiles
? If you pass header metadata (which I guess is any metadata?) it sets headers. If you don't want to set headers, don't pass metadata.
As is, there's effectively two implementations of Publish methods and 3 layers of duplicated code leading into S3Client.PutFile()
or S3Client.PutFileWithHeaders()
which are 98% identical.
Perhaps could deprecate the WithHeaders methods, leaving them in (inheriting and identical), but I get the sense this isn't heavily used enough that this would be a huge deal..?
Didn't realize but there's actually an AppVeyor build setup: https://ci.appveyor.com/project/makomweb/emkay-s3
That should be linked in the README so it can be found from Github, plus it would be good to put a build status badge in.
A declarative, efficient, and flexible JavaScript library for building user interfaces.
๐ Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.
TypeScript is a superset of JavaScript that compiles to clean JavaScript output.
An Open Source Machine Learning Framework for Everyone
The Web framework for perfectionists with deadlines.
A PHP framework for web artisans
Bring data to life with SVG, Canvas and HTML. ๐๐๐
JavaScript (JS) is a lightweight interpreted programming language with first-class functions.
Some thing interesting about web. New door for the world.
A server is a program made to process requests and deliver data to clients.
Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.
Some thing interesting about visualization, use data art
Some thing interesting about game, make everyone happy.
We are working to build community through open source technology. NB: members must have two-factor auth.
Open source projects and samples from Microsoft.
Google โค๏ธ Open Source for everyone.
Alibaba Open Source for everyone
Data-Driven Documents codes.
China tencent open source team.