EmranAhmed / woo-product-download-from-amazon-s3

WooCommerce Product Download / Upload to / from using Amazon S3 service.
https://wordpress.org/plugins/woo-product-download-from-amazon-s3/
GNU General Public License v3.0
6 stars 3 forks source link

What about v4? #2

Open GsHeri opened 7 years ago

GsHeri commented 7 years ago

I'd really love this plugin, if it could sign requests with v4 (frankfurt for example). right now, this plugin is useless. all it takes would be v4 signing to work... could you please implement it?

marbetschar commented 7 years ago

Same error here - any news?

PHP Warning:  S3::listBuckets(): [SignatureDoesNotMatch] The request signature we calculated does not match the signature you provided. Check your key and signing method. in /app/data/wp-content/plugins/woo-product-download-from-amazon-s3/includes/S3.php on line 373
marbetschar commented 7 years ago

FWIW: Is it possible to get a quick fix for this one? I know you mentioned a major rewrite and I'm happy to help with it.

But for the current project I need a solution quite fast and your plugin does exactly what I was hoping for - not like the other plugins out there. So I would truly appreciate using your plugin if possible at all.

marbetschar commented 7 years ago

For those who this is also an issue, you can grab a copy of my created fork which supports Amazon S3 v4 here: https://github.com/Mandelkind/woo-product-download-from-amazon-s3/tree/minio

Please note I only tested it against my own minio instance, but since it is backed by the official Amazon S3 PHP SDK you should be good to go!

Looking forward to the new rewritten version of this plugin though.

EmranAhmed commented 7 years ago

@marbetschar Could you please send a PR? Rewritten version will come with VueJS. But now people need a solution, Please mention your wp user name also :)

Thank you :)

EmranAhmed commented 7 years ago

BTW, Does your edited script supports us-east-2 region?

Lists of all regions: http://docs.aws.amazon.com/general/latest/gr/rande.html#s3_region