Skip to content
This repository has been archived by the owner on Jul 13, 2023. It is now read-only.

Update s3.rb #2444

Closed
wants to merge 1 commit into from
Closed

Update s3.rb #2444

wants to merge 1 commit into from

Conversation

Daniel-ltw
Copy link

AWS-SDK has change their config

They have stop using proxy_uri
https://github.com/aws/aws-sdk-ruby/blob/8ff3f1cedbac5820336c4df3f5d819ce8de6e1f6/aws-sdk-core/lib/seahorse/client/net_http/connection_pool.rb#L19

The seahorse client uses http_proxy.

One could overwrite paperclip setting by manually setting this with the AWS SDK gem, by doing:
AWS.config.update({
http_proxy: 'http://blah.blah.blah:8080'
})

Paperclip should stop doing config[:proxy_uri]

AWS-SDK has change their config
@jeremywadsack
Copy link
Contributor

Isn't this a duplicate of #2438?

@Daniel-ltw
Copy link
Author

Assuming that it has been fixed in #2438, it should be merge?

@jeremywadsack
Copy link
Contributor

Agreed. One should be merged, the other should be closed. I'm not sure what's going on with that: #2477

@mike-burns
Copy link
Contributor

We have switched to aws-sdk-s3.

@mike-burns mike-burns closed this Apr 27, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants