fix deleting array content when passing an array as payload

Author: carlastabile

lib/auth0/mixins/httpproxy.rb

@@ -18,7 +18,8 @@ module HTTPProxy
       # proxying requests from instance methods to HTTP class methods
       %i(get post post_file post_form put patch delete delete_with_body).each do |method|
         define_method(method) do |uri, body = {}, extra_headers = {}|
-          body = body.delete_if { |_, v| v.nil? }
+          # body = body.delete_if { |_, v| v.nil? }
+          body = body.dup.delete_if { |_, v| v.nil? } if body.is_a?(Hash)
           token = get_token()
           authorization_header(token) unless token.nil?
           request_with_retry(method, uri, body, extra_headers)

spec/lib/auth0/mixins/httpproxy_spec.rb

@@ -279,6 +279,19 @@ def expected_payload(method, overrides = {})
         expect { @instance.send(http_method, '/test') }.not_to raise_error
       end
 
+      it "should handle array parameters for #{http_method} method" do
+        array_data = ['param1', 'param2']
+        if http_method == :post_form
+          expected_params = expected_payload(http_method, { payload: array_data })
+        else
+          expected_params = expected_payload(http_method, { payload: array_data.to_json })
+        end
+
+        expect(RestClient::Request).to receive(:execute).with(expected_params)
+          .and_return(StubResponse.new({}, true, 200))
+        expect { @instance.send(http_method, '/test', array_data) }.not_to raise_error
+      end
+
       it 'should not raise exception if data returned not in json format (should be fixed in v2)' do
         allow(RestClient::Request).to receive(:execute).with(expected_payload(http_method))
           .and_return(StubResponse.new('Some random text here', true, 200))