Commit 6e0c9e99 by Jon Yurek

Revert "S3 Regions in the cucubmer specs"

Turns out it didn't fix test failures like I thought. It only added
more!

This reverts commit b8221c6b.
parent 5c43151d
......@@ -77,5 +77,5 @@ Feature: Rails integration
And I attach the file "spec/support/fixtures/5k.png" to "Attachment" on S3
And I press "Submit"
Then I should see "Name: something"
And I should see an image with a path of "http://s3-us-west-2.amazonaws.com/paperclip/attachments/original/5k.png"
And the file at "http://s3-us-west-2.amazonaws.com/paperclip/attachments/original/5k.png" should be uploaded to S3
And I should see an image with a path of "http://s3.amazonaws.com/paperclip/attachments/original/5k.png"
And the file at "http://s3.amazonaws.com/paperclip/attachments/original/5k.png" should be uploaded to S3
......@@ -209,12 +209,11 @@ module Paperclip
host_name = @options[:s3_host_name]
host_name = host_name.call(self) if host_name.is_a?(Proc)
region = ["s3", s3_region].compact.join("-")
host_name || s3_credentials[:s3_host_name] || "#{region}.amazonaws.com"
host_name || s3_credentials[:s3_host_name] || "s3.amazonaws.com"
end
def s3_region
region = @options[:s3_region] || s3_credentials[:s3_region]
region = @options[:s3_region]
region = region.call(self) if region.is_a?(Proc)
region || s3_credentials[:s3_region]
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment