qoobaa/s3

Wrong params order in expiring_url for paperclip

ace opened this issue · 2 comments

ace commented

s3_paperclip.rb (to be copied as an initalizer) defines expiring_url as


but in paperclip's s3.rb it is defined as

So, trying to use something like

yields a url with an expiration of 3600, not 10.

Reversing the params order in the initializer just solves the problem. It would be nice to have it reversed also in s3_paperclip.rb so it can be copied as is.

Can you create a patch?

I'll move paperclip integration to a separate gem.