carrierwave_direct 0.0.3 → 0.0.4

data/README.md

@@ -374,3 +374,4 @@ You should now be able to run the tests
 
 * cblunt (Chris Blunt) - Support for passing html options
 * robyurkowski (Rob Yurkowski) - Fix deprecation warnings for Rails 3.2
+* tylr (Tyler Love) - Bug fix

data/lib/carrierwave_direct/uploader.rb

@@ -24,7 +24,7 @@ module CarrierWaveDirect
       if options[:with_path]
         uri = URI.parse(fog_uri)
         path = "/#{key}"
-        uri.path = path
+        uri.path = URI.escape(path)
         fog_uri = uri.to_s
       end
       fog_uri
@@ -135,4 +135,3 @@ module CarrierWaveDirect
     end
   end
 end
-

data/lib/carrierwave_direct/version.rb

@@ -1,6 +1,6 @@
 # encoding: utf-8
 
 module CarrierwaveDirect
-  VERSION = "0.0.3"
+  VERSION = "0.0.4"
 end
 

data/spec/uploader_spec.rb

@@ -8,6 +8,7 @@ describe CarrierWaveDirect::Uploader do
 
   SAMPLE_DATA = {
     :path => "upload_dir/bliind.exe",
+    :path_with_special_chars => "upload_dir/some file & blah.exe",
     :key => "some key",
     :guid => "guid",
     :store_dir => "store_dir",
@@ -215,6 +216,15 @@ describe CarrierWaveDirect::Uploader do
     end
 
     context ":with_path => true" do
+
+      context "#key is set to '#{sample(:path_with_special_chars)}'" do
+        before { subject.key = sample(:path_with_special_chars) }
+
+        it "should return the full url with '/#{URI.escape(sample(:path_with_special_chars))}' as the path" do
+          URI.parse(subject.direct_fog_url(:with_path => true)).path.should == "/#{URI.escape(sample(:path_with_special_chars))}"
+        end
+      end
+
       context "#key is set to '#{sample(:path)}'" do
         before { subject.key = sample(:path) }
 
@@ -454,4 +464,3 @@ describe CarrierWaveDirect::Uploader do
     end
   end
 end
-

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: carrierwave_direct
 version: !ruby/object:Gem::Version
-  version: 0.0.3
+  version: 0.0.4
   prerelease: 
 platform: ruby
 authors:
@@ -9,11 +9,11 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2012-02-06 00:00:00.000000000Z
+date: 2012-02-08 00:00:00.000000000Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: carrierwave
-  requirement: &85732130 !ruby/object:Gem::Requirement
+  requirement: &72575860 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -21,10 +21,10 @@ dependencies:
         version: '0'
   type: :runtime
   prerelease: false
-  version_requirements: *85732130
+  version_requirements: *72575860
 - !ruby/object:Gem::Dependency
   name: uuid
-  requirement: &85731800 !ruby/object:Gem::Requirement
+  requirement: &72575510 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -32,10 +32,10 @@ dependencies:
         version: '0'
   type: :runtime
   prerelease: false
-  version_requirements: *85731800
+  version_requirements: *72575510
 - !ruby/object:Gem::Dependency
   name: fog
-  requirement: &85731490 !ruby/object:Gem::Requirement
+  requirement: &72575190 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -43,10 +43,10 @@ dependencies:
         version: '0'
   type: :runtime
   prerelease: false
-  version_requirements: *85731490
+  version_requirements: *72575190
 - !ruby/object:Gem::Dependency
   name: rspec
-  requirement: &85731170 !ruby/object:Gem::Requirement
+  requirement: &72574880 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -54,10 +54,10 @@ dependencies:
         version: '0'
   type: :development
   prerelease: false
-  version_requirements: *85731170
+  version_requirements: *72574880
 - !ruby/object:Gem::Dependency
   name: timecop
-  requirement: &85730900 !ruby/object:Gem::Requirement
+  requirement: &72574590 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -65,10 +65,10 @@ dependencies:
         version: '0'
   type: :development
   prerelease: false
-  version_requirements: *85730900
+  version_requirements: *72574590
 - !ruby/object:Gem::Dependency
   name: rails
-  requirement: &85730610 !ruby/object:Gem::Requirement
+  requirement: &72574320 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -76,10 +76,10 @@ dependencies:
         version: '0'
   type: :development
   prerelease: false
-  version_requirements: *85730610
+  version_requirements: *72574320
 - !ruby/object:Gem::Dependency
   name: sqlite3
-  requirement: &85730290 !ruby/object:Gem::Requirement
+  requirement: &72574040 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -87,10 +87,10 @@ dependencies:
         version: '0'
   type: :development
   prerelease: false
-  version_requirements: *85730290
+  version_requirements: *72574040
 - !ruby/object:Gem::Dependency
   name: capybara
-  requirement: &85729900 !ruby/object:Gem::Requirement
+  requirement: &72573710 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -98,7 +98,7 @@ dependencies:
         version: '0'
   type: :development
   prerelease: false
-  version_requirements: *85729900
+  version_requirements: *72573710
 description: Process your uploads in the background by uploading directly to S3
 email:
 - dwilkie@gmail.com
@@ -156,7 +156,7 @@ required_ruby_version: !ruby/object:Gem::Requirement
       version: '0'
       segments:
       - 0
-      hash: -831805485
+      hash: 386717071
 required_rubygems_version: !ruby/object:Gem::Requirement
   none: false
   requirements:
@@ -165,7 +165,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
       version: '0'
       segments:
       - 0
-      hash: -831805485
+      hash: 386717071
 requirements: []
 rubyforge_project: carrierwave_direct
 rubygems_version: 1.8.10