jnicklas/carrierwave — pull request #685 by adrianpike (2012-04-04 19:32:38+00:00)
Title: Escape plusses for the download!

If there's a plus in, say, an S3 signed request generated by fog, and it gets unescaped, you'll get a 403 when you try and hit it.

I was getting similar symptoms to #293 whenever my signed requests to S3 had a plus in them - this fixes that. I'd like to make sure it doesn't break urls for folks if there's a plus in the non-params.

Another possibility (and one that I'm actually doing on my side of things) is to get smarter about how params are handled in a remote_url. I'm having to explicitly strip off params in [my fork of carrierwave_direct](https://github.com/adrianpike/carrierwave_direct/blob/c73c79344444a65f7adffa034b8696d8fa5c36e0/lib/carrierwave_direct/uploader.rb#L94), otherwise my generated versions have the params included in their name. This is on my list of things to investigate post-launch, whether this is cw-direct or cw's responsibility.

Thanks!