From b5da9419aff93bc1f063417046cf8838bc0f61ab Mon Sep 17 00:00:00 2001 From: Bernard Grymonpon Date: Mon, 25 Jun 2018 14:05:52 +0200 Subject: [PATCH 1/2] Pass the options as params to request The options var should be injected as extra_params, not as a body-content. --- lib/dogapi/v1/hosts.rb | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/lib/dogapi/v1/hosts.rb b/lib/dogapi/v1/hosts.rb index 50c180a4..7bd65aaf 100644 --- a/lib/dogapi/v1/hosts.rb +++ b/lib/dogapi/v1/hosts.rb @@ -9,7 +9,7 @@ class HostsService < Dogapi::APIService API_VERSION = 'v1' def search(options = {}) - request(Net::HTTP::Get, "/api/#{API_VERSION}/hosts", nil, options, true) + request(Net::HTTP::Get, "/api/#{API_VERSION}/hosts", options, nil, true) end def totals From 506aefb3a28ac5cbb829ab18d6ca0f1b27d1d6e4 Mon Sep 17 00:00:00 2001 From: Bernard Grymonpon Date: Wed, 27 Jun 2018 13:44:11 +0200 Subject: [PATCH 2/2] Removes the pp requirement The include statement installs https://rubygems.org/gems/pp/versions/0.1.1, which is not what we want... --- lib/dogapi/common.rb | 1 - 1 file changed, 1 deletion(-) diff --git a/lib/dogapi/common.rb b/lib/dogapi/common.rb index fd626eae..63f312cc 100644 --- a/lib/dogapi/common.rb +++ b/lib/dogapi/common.rb @@ -1,6 +1,5 @@ require 'cgi' require 'net/https' -require 'pp' require 'socket' require 'uri'