lumberg 3.0.0 → 3.1.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 62bbbbeac1e7b971a4d258310f24dc22e5d4bca2
-  data.tar.gz: 8a451da0ae4001a4cd9d9415b2edca146d683f53
+  metadata.gz: 52c46d1297e939a78ed704799eb7c2f7f5dfd0ac
+  data.tar.gz: 1fa9469f31285e6feed8453e373d1a292be4e7a7
 SHA512:
-  metadata.gz: 9798a4a18f6e7603daba5a8ceb878a3ef8448deff7b31655a177070c14c970f69194204ddc0ea6ca419f0f2e4f1074e5f0d9351eac6e13d4e7d315602bb7e977
-  data.tar.gz: 5ddb8c7a7a0e9b020a2ac75c18629246922fce6cf9f9fa3b7cd6669ee8130aab6d387c593cdb8cef5eebac735aa81509d91611c34dd91102f8d10f639fc4b365
+  metadata.gz: 97879e4cf0e072b5887441a2f08cee3154d14573c2318cb75bc352d715a46e93a4a08912a639911329902c2bab403619d9cd9635b1ed3788991b69b7fde5742d
+  data.tar.gz: 85b21775cfd5c1a9b795c835429d434899cffe9440373a0267449793c5723562e1329b53bf7b3117ac9b2328d4b8becdf1f004efa5f385420815b6dbb3552faf

data/lib/lumberg/format_whm.rb

@@ -26,7 +26,15 @@ module Lumberg
                env[:body]
              end
 
-      env[:body] = format_response body
+      if body =~ /cPanel operations have been temporarily suspended/
+        raise Lumberg::WhmConnectionError.new(body)
+      end
+
+      if @type == :whostmgr || response_type(body) == :whostmgr
+        env[:body] = format_response body
+      else
+        env[:body] = format_response JSON.parse(body)
+      end
     end
 
     def response_values(env)

data/lib/lumberg/version.rb

@@ -1,3 +1,3 @@
 module Lumberg
-  VERSION = '3.0.0'
+  VERSION = '3.1.0'
 end

data/lib/lumberg/whm/server.rb

@@ -251,7 +251,6 @@ module Lumberg
           c.request :url_encoded
           c.response :format_whm, @force_response_type, @response_key, @boolean_params
           c.response :logger, create_logger_instance
-          c.response :json unless @force_response_type == :whostmgr
           c.adapter :net_http
           c.options[:timeout] = timeout if timeout
         end.get(function).body

data/spec/format_whm_spec.rb

@@ -6,8 +6,9 @@ describe Lumberg::FormatWhm do
 
   context "gzip" do
     let(:gzipped_data) do
-      "\u001F\x8B\b\u0000\u001E¹R\u0000\u0003+\xCE\xCFMUHI,I\u0004\u0000" +
-      "\u001E\xE9\xC2\xD9\t\u0000\u0000\u0000"
+      "\x1F\x8B\b\x00\x13d\x01V\x00\x03\xABV*.MNN-.V\xB2*)*M\xD5Q\xCA" +
+      "\x05\xB2\x13\xD3S\x95\xAC\x94\x8A\xF3sS\x15R\x12K\x12\x95j\x01" +
+      "\xA0E\x91\xF9&\x00\x00\x00"
     end
 
     it "removes content encoding" do
@@ -18,13 +19,14 @@ describe Lumberg::FormatWhm do
 
       env.should_not include "content-encoding"
 
-      env[:body][:params].should eq "some data"
+      env[:body][:params][:message].should eq "some data"
     end
   end
 
   context "deflate" do
     let(:compressed_data) do
-      "x\x9C+\xCE\xCFMUHI,I\x04\x00\x11\x81\x03o"
+      "x\x9C\xABV*.MNN-.V\xB2*)*M\xD5Q\xCA\x05\xB2\x13\xD3S\x95\xAC" +
+      "\x94\x8A\xF3sS\x15R\x12K\x12\x95j\x01\n\xFE\rq"
     end
 
     it "removes content encoding" do
@@ -35,19 +37,28 @@ describe Lumberg::FormatWhm do
 
       env.should_not include "content-encoding"
 
-      env[:body][:params].should eq "some data"
+      env[:body][:params][:message].should eq "some data"
     end
   end
 
   context "no content encoding" do
     it "doesn't touch response headers" do
-      env = { body: "some data", response_headers: { foo: "bar" } }
+      env = { body: "{\"success\":true,\"message\":\"some data\"}",
+              response_headers: { foo: "bar" } }
 
       subject.on_complete(env)
 
       env[:response_headers].should include :foo
 
-      env[:body][:params].should eq "some data"
+      env[:body][:params][:message].should eq "some data"
+    end
+  end
+
+  context "API unavailable" do
+    it "raises a Lumberg::WhmConnectionError" do
+      env = { body: "cPanel operations have been temporarily suspended", response_headers: { foo: "bar" } }
+
+      expect { subject.on_complete(env) }.to raise_error(Lumberg::WhmConnectionError)
     end
   end
 end

data/spec/whm/server_spec.rb

@@ -164,6 +164,7 @@ module Lumberg
 
           before(:each) do
             Net::HTTP.stub(:new) { net_http }
+            JSON.stub(:parse) { { } }
           end
 
           it "sets HTTP timeout when assigned" do

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: lumberg
 version: !ruby/object:Gem::Version
-  version: 3.0.0
+  version: 3.1.0
 platform: ruby
 authors:
 - Justin Mazzi
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2015-08-12 00:00:00.000000000 Z
+date: 2015-09-22 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: json