martinklepsch / s3-beam

πŸš€ direct-to-S3 uploading using ClojureScript
Eclipse Public License 1.0
92 stars 17 forks source link

aws-zone not required when passing custom upload-url #44

Closed benhowell closed 6 years ago

benhowell commented 6 years ago

I'm using s3-beam for DigitalOcean spaces and therefore using custom upload-url. However, the handler checks for aws-zone and bombs out if it's not there.

As a work-around, you can set any arbitrary aws zone (e.g. "us-east-1") as it's not used in the hash or anything.

Thanks!

martinklepsch commented 6 years ago

Hey @benhowell, that's a good point β€” would you be interested in making a PR? I'm thinking the following might be helpful for you and others:

Would be happy about your contribution πŸ™Œ

benhowell commented 6 years ago

Sure, I'll put it on my todo list! I'll leave the issue open until I fix it :+1:

benhowell commented 6 years ago

Done :-)

benhowell commented 6 years ago

Reopening until pulled or otherwise

martinklepsch commented 6 years ago

Haven’t cut a release yet but will try to do so later today. Thanks :) On Sun 26. Nov 2017 at 03:37, Ben Howell notifications@github.com wrote:

Closed #44 https://github.com/martinklepsch/s3-beam/issues/44.

β€” You are receiving this because you commented.

Reply to this email directly, view it on GitHub https://github.com/martinklepsch/s3-beam/issues/44#event-1358014714, or mute the thread https://github.com/notifications/unsubscribe-auth/AAF82NU7o-sXUNoC0eZLBbb3EYwK-pukks5s6M7SgaJpZM4PoRIl .

martinklepsch commented 6 years ago

Alright I cut 0.6.0-alpha4 now β€” I stripped down the Digital Ocean specific documentation quite a bit, feeling that people are best of just referring to the official docs.

benhowell commented 6 years ago

Thanks! :+1: