carrierwaveuploader/carrierwave — pull request #1332 by leods92 (2014-03-06 19:18:36+00:00)
Title: Fixed #public_url to generate proper HTTPS URL.

I'm don't know all the ins and outs of CarrierWave and Fog but I know that:
* CarrierWave generates public URLs itself
* Fog also does that in a more thorough way

The problem is CarrierWave public URL generation doesn't consider a HTTPS scenario.
One should not use it with subdomains or custom domains.
Browsers raise security warnings if we do.

That said, I'm suggesting in this PR to use the "directory" URL syntax when HTTPS and custom domains are required.
Anyway, I think this doesn't look DRY and we should leave URL generation to Fog.

What do you guys think?
Maybe I'm missing something here...