flexirest 1.3.34 → 1.3.35

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 0aefbb07885064fa794f6680446e48a678682a6a
-  data.tar.gz: dae31dfd7a9522ac6e5f21b72037b1d4be9ed47c
+  metadata.gz: e6b067ad0cc1312a876a41486f10d6368b3b34e0
+  data.tar.gz: e2dd9f61d27a472b42a05d7798b752fcf5682da0
 SHA512:
-  metadata.gz: 39aa00c5bc3ff7bce1ef08cfa646bf371dbeff2f13464c750cf462d2ee93cedbe90be9c4b3fa77788240c7a186568de55df382a7c287287fcd044dcc2dbfd49a
-  data.tar.gz: 56f5bdabdfb819ea43a2ea209700d330d05548e174ef3343c5e7cc15a00b69fd6b0ba6c705502263609d03481b72812204632f141bd6f1a90a72ac120fdc2cf4
+  metadata.gz: 22af0afd6a7bbd09b14a7e816cd87cce96a966eac6495d3a6e78ee47532e000772261ebac173c9ebda037b1b306dd75a9aa06df7e2dbd48690591d17ca729bf1
+  data.tar.gz: 6bf3a1d2cae10e37486bca474a91d3cdf74a44d7183b3dd2620a0abee736955e147030b2f6b740074b977115c790419679933b657f78a3da1f8c051d2d8d2892

data/CHANGELOG.md

@@ -1,5 +1,11 @@
 # Changelog
 
+## 1.3.35
+
+Bugfix:
+
+- Slashes in URL params are now escaped to %2F (thanks to davidtolsma for the bug report)
+
 ## 1.3.34
 
 Feature:

data/lib/flexirest/request.rb

@@ -307,7 +307,7 @@ module Flexirest
           target = @get_params.delete(token.to_sym) || @post_params.delete(token.to_sym) || @get_params.delete(token.to_s) || @post_params.delete(token.to_s) || ""
           # it's possible the URL path variable may not be part of the request, in that case, try to resolve it from the object attributes
           target = @object._attributes[token.to_sym] || "" if target == ""
-          @url.gsub!(":#{token}", URI.escape(target.to_s))
+          @url.gsub!(":#{token}", URI.escape(target.to_s).gsub("/", "%2F"))
         end
       end
     end

data/lib/flexirest/version.rb

@@ -1,3 +1,3 @@
 module Flexirest
-  VERSION = "1.3.34"
+  VERSION = "1.3.35"
 end

data/spec/lib/request_spec.rb

@@ -200,6 +200,11 @@ describe Flexirest::Request do
     ExampleClient.find id:"foo bar"
   end
 
+  it "should pass URL-encode URL parameters including slashes" do
+    expect_any_instance_of(Flexirest::Connection).to receive(:get).with("/foo%2Fbar", an_instance_of(Hash)).and_return(::FaradayResponseMock.new(OpenStruct.new(body:'{"result":true}', response_headers:{})))
+    ExampleClient.find id:"foo/bar"
+  end
+
   it "should accept an integer as the only parameter and use it as id" do
     expect_any_instance_of(Flexirest::Connection).to receive(:get).with("/1234", an_instance_of(Hash)).and_return(::FaradayResponseMock.new(OpenStruct.new(body:'{"result":true}', response_headers:{})))
     ExampleClient.find(1234)

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: flexirest
 version: !ruby/object:Gem::Version
-  version: 1.3.34
+  version: 1.3.35
 platform: ruby
 authors:
 - Andy Jeffries
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2017-04-27 00:00:00.000000000 Z
+date: 2017-06-13 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: bundler