Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Please add clear documentation for s3 regions #88

Closed
codecowboy opened this issue Apr 26, 2014 · 3 comments
Closed

Please add clear documentation for s3 regions #88

codecowboy opened this issue Apr 26, 2014 · 3 comments

Comments

@codecowboy
Copy link

I had hours of pain getting the S3 adapter talking to the eu-west-1 region.

Please add detailed instructions for setting regions in the documentation.

http://stackoverflow.com/questions/23307332/error-when-attempting-upload-to-s3-with-gaufrette/23307443#23307443

@sc0rp10
Copy link

sc0rp10 commented Sep 29, 2015

+1
Also, there is an error with credentials:

services:
    acme.aws_s3.client:
        class: Aws\S3\S3Client
        factory_class: Aws\S3\S3Client
        factory_method: 'factory'
        arguments:
            -
                key: %amazon_s3.key%
                secret: %amazon_s3.secret%
                region: %amazon_s3.region%

Isn't correct; correct config looks like:

services:
    acme.aws_s3.client:
        class: Aws\S3\S3Client
        factory_class: Aws\S3\S3Client
        factory_method: 'factory'
        arguments:
            -
                credentials: # <-----
                    key: %amazon_s3.key%
                    secret: %amazon_s3.secret%

at https://github.com/KnpLabs/KnpGaufretteBundle#defining-services-1

@acanimal
Copy link

Almost a couple of years later but... for me the configuration specified at docs works (https://github.com/KnpLabs/KnpGaufretteBundle/tree/0.3.0#awss3) and also I found the list of regions and their names at: http://docs.aws.amazon.com/general/latest/gr/rande.html#s3_region

@akerouanton
Copy link
Contributor

Thank your @codecowboy for the report. I added a little note about regions in the documentation.

@sc0rp10 This has been fixed too :)

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

No branches or pull requests

4 participants