flexirest 1.6.8 → 1.6.9

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 48c8e263cf0b1c959599454ce360b612b279b900
-  data.tar.gz: eba90a77d94f9d8e29f651ec1a1b7815a10b4aa6
+  metadata.gz: 97942021a18075e99ef0ec8b44ddcfc65666ccf2
+  data.tar.gz: 0e65b5b0d7cc7ce9248118eb2d87bde8a4cb51d0
 SHA512:
-  metadata.gz: a43bc7600fd621a71e595e55b1aebddc19feda4eae959a376c429eb4498e35d2f55c64c62c5a97f4769a98036d8f31a207da90e72de94133d38a5ad5c680b080
-  data.tar.gz: 895458b577307cf2a2be5681fb827979847d61b13e36f89c236a3c1ddffcf3f98ace941fa9ed639e7cb7287ea5d291f0433f4ed8f360f5e7c305507e02853e4c
+  metadata.gz: 83f1b98ee42cc11c26edb512b85e30c050affd2d9efed3d3ccaa9a497c342aa5690f5f0ea16be3c18a99a5edb5657d97c2fc4ef0a8ffbdfe3edda590e174c6d2
+  data.tar.gz: d61bd29b62610ad7e852d843391f83c6f19d0fa5ab427c9993264c875e76d7b97442fc5ad60a3f3666de2f204e67321d0e2a2833a9e4067f4bdaa23cffa1d69d

data/CHANGELOG.md

@@ -1,5 +1,11 @@
 # Changelog
 
+## 1.6.9
+
+Feature:
+
+- Empty response bodies with an unparseable content-type are forced to be JSON with an empty JSON body
+
 ## 1.6.8
 
 Feature:

data/lib/flexirest/request.rb

@@ -524,7 +524,10 @@ module Flexirest
     def handle_response(response, cached = nil)
       @response = response
       status = @response.status || 200
-      @response.body = "{}" if @response.body.blank?
+      if @response.body.blank?
+        @response.response_headers['Content-Type'] = "application/json"
+        @response.body = "{}"
+      end
 
       if cached && response.status == 304
         Flexirest::Logger.debug "  \033[1;4;32m#{Flexirest.name}\033[0m #{@instrumentation_name}" +

data/lib/flexirest/version.rb

@@ -1,3 +1,3 @@
 module Flexirest
-  VERSION = "1.6.8"
+  VERSION = "1.6.9"
 end

data/spec/lib/request_spec.rb

@@ -430,6 +430,11 @@ describe Flexirest::Request do
     expect(ExampleClient.all).to be_truthy
   end
 
+  it "should return true from 200 with empty bodies even if they don't have a correct content type" do
+    expect_any_instance_of(Flexirest::Connection).to receive(:get).with(any_args).and_return(::FaradayResponseMock.new(OpenStruct.new(status:200, response_headers:{"Content-Type" => "text/plain"}, body: nil)))
+    expect(ExampleClient.all).to be_truthy
+  end
+
   it "should return a lazy loader object if lazy loading is enabled" do
     object = LazyLoadedExampleClient.fake id:1234, debug:true
     expect(object).to be_an_instance_of(Flexirest::LazyLoader)

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: flexirest
 version: !ruby/object:Gem::Version
-  version: 1.6.8
+  version: 1.6.9
 platform: ruby
 authors:
 - Andy Jeffries
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2018-06-11 00:00:00.000000000 Z
+date: 2018-07-24 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: bundler