Skip to content

Loading…

Upload url bug http:/ not http:// #19

Merged
merged 1 commit into from

3 participants

@clemensg

poke

@rmm5t
CarrierWave member

Can we get a test for this fix?

@rmm5t rmm5t merged commit 4765abd into carrierwaveuploader:master
@rmm5t
CarrierWave member

@huacnlee Thanks for this! In addition to your fix, I also added some extra tests to both cover your use case and to prevent regressions of possibly doubling up path separators in the URL path.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Commits on Oct 26, 2011
  1. @huacnlee
Showing with 1 addition and 1 deletion.
  1. +1 −1 lib/carrierwave/storage/grid_fs.rb
View
2 lib/carrierwave/storage/grid_fs.rb
@@ -52,7 +52,7 @@ def url
unless @uploader.grid_fs_access_url
nil
else
- [@uploader.grid_fs_access_url, @path].join("/").squeeze("/")
+ [@uploader.grid_fs_access_url, @path].join("/")
end
end
Something went wrong with that request. Please try again.