tpett/perry — pull request #12 by kellyredding (2012-08-08 21:41:33+00:00)
Title: restful http adapter: have any `:default_options` placed in the query string

## The Problem

Currently, the adapter is smart and detects if the type of request
you are submitting has a post body (:post, :put) it will place the
options in the post body and the request doesn't (:get, :delete)
places them in the query string. Very cool, except...

This means where your options are placed in the request is
inconsistant depending on the type of request you are submitting.
This makes it difficult to programmatically inspect the request when the
thing doing the inspecting either doesn't have access to the post body
(HAProxy for example) or the framework you are using doesn't combine
the post body params with the query params into a single params object
(like Rails does).

## Proposed Change

Always place any `:default_options` in the query string, regardless of the
type of request.

## Results

Updated the tests (mostly registering new FakeWeb endpoints with the query
string info) and all are passing. I'll post diff comments to explain my changes
and reasoning.

Thank man!