diff --git a/lib/serpapi/client.rb b/lib/serpapi/client.rb index ff1e583..0361d20 100644 --- a/lib/serpapi/client.rb +++ b/lib/serpapi/client.rb @@ -229,12 +229,12 @@ def get(endpoint, decoder = :json, params = {}) # parse JSON response with Ruby standard library data = JSON.parse(response.body, symbolize_names: symbolize_names) if data.instance_of?(Hash) && data.key?(:error) - raise SerpApiError, "HTTP request failed with error: #{data[:error]} from url: https://#{BACKEND}#{endpoint}, params: #{params}, decoder: #{decoder}, response status: #{response.status} " + raise SerpApiError, "HTTP request failed with error: #{data[:error]}, from url: https://#{BACKEND}#{endpoint}, params: #{params}, decoder: #{decoder}, response status: #{response.status} " elsif response.status != 200 - raise SerpApiError, "HTTP request failed with response status: #{response.status} reponse: #{data} on get url: https://#{BACKEND}#{endpoint}, params: #{params}, decoder: #{decoder}" + raise SerpApiError, "HTTP request failed with response status: #{response.status}, response: #{data} on get url: https://#{BACKEND}#{endpoint}, params: #{params}, decoder: #{decoder}" end rescue JSON::ParserError - raise SerpApiError, "JSON parse error: #{response.body} on get url: https://#{BACKEND}#{endpoint}, params: #{params}, decoder: #{decoder}, response status: #{response.status}" + raise SerpApiError, "JSON parse error: #{response.body}, on get url: https://#{BACKEND}#{endpoint}, params: #{params}, decoder: #{decoder}, response status: #{response.status}" end # discard response body @@ -244,7 +244,7 @@ def get(endpoint, decoder = :json, params = {}) when :html # html decoder if response.status != 200 - raise SerpApiError, "HTTP request failed with response status: #{response.status} reponse: #{data} on get url: https://#{BACKEND}#{endpoint}, params: #{params}, decoder: #{decoder}" + raise SerpApiError, "HTTP request failed with response status: #{response.status}, response: #{data}, on get url: https://#{BACKEND}#{endpoint}, params: #{params}, decoder: #{decoder}" end response.body diff --git a/spec/serpapi/client/client_spec.rb b/spec/serpapi/client/client_spec.rb index a5f395c..ab33d35 100644 --- a/spec/serpapi/client/client_spec.rb +++ b/spec/serpapi/client/client_spec.rb @@ -90,7 +90,7 @@ begin client.send(:get, '/invalid', :html, {}) rescue SerpApi::SerpApiError => e - expect(e.message).to include(/HTTP request failed with response status: 404 Not Found reponse/), "got #{e.message}" + expect(e.message).to include(/HTTP request failed with response status: 404 Not Found response/), "got #{e.message}" rescue => e raise("wrong exception: #{e}") end