zpatrick/rclient

Minimalistic REST client for Go applications

rclient's Issues

error parsing should be easier

Right now, you need to totally override the request reader if you want to read custom error types. It should be more simple to try and marshal into errors. Perhaps a request option:

var resp int
var myerr *MyCustomError
client.Get("/path", &resp, rclient.Error(&myerr))

Another option could map status codes to types:

client.Get("/path", &resp, rclient.StatusMap(
    series.Ints(200, 299): &someType,
    series.Ints(400, 499): &someError,
    series.Ints(500, 599): &someOtherError,
))

Add http.Response to ReadJSONResponse error

The current error in ReadJSONResponse isn't very useful when trying to handle errors:

return fmt.Errorf("Invalid status code: %d", resp.StatusCode)

This should return a custom error type that has the http.Response as a public field.

readme should be less verbose

Instead of going over each request/client option, I should have a slimmed down version in the readme and examples of each option in godoc, e.g.:

# Request Options
* [Header](#link-to-example) - Add a header to a request
* [Basic Auth](#link-to-example) - Add basic auth  to a request

# Client Options
* [Doer](#) - Specify which object performs the http.Request (the default is http.DefaultClient)

# Error Handling
## using rcilent.ResponseError
## using ResponseReader() option