gosexy/rest

The rest package helps you creating clients for HTTP APIs with Go.

rest's Issues

Method returning io.Reader

Hello,

maybe you could add method returning body as io.Reader.

What for? Well, html package http://godoc.org/code.google.com/p/go.net/html (moved here from exp/html) for Parse request us to provide io.Reader.

Now we need to:

import "bytes"
import "code.google.com/p/go.net/html"
...
rawDoc, _ := client.Get(values).Do()
// Parse the HTML into nodes
root, err := html.Parse(bytes.NewReader(rawDoc))

Maybe something like this would be shorter:

import "code.google.com/p/go.net/html"
...
rawDoc, _ := client.Get(values).Body()
// Parse the HTML into nodes
root, _ := html.Parse(rawDoc)

Best regards,
Dobrosław Żybort

Don't add / to requested url end

Hello,

if I try do get image, library will add unnecessary / to requested url.

package main

import(
    "fmt"

    "github.com/gosexy/rest"
)

func main(){
    url := "http://stackoverflow.com/favicon.ico"
    client := rest.New(url)

    // Get URL response body
    _, err := client.Get(nil).Do()
    fmt.Println("Request url:", client.Request.URL.String())
    if err != nil {
        fmt.Println(err)
    }
    if client.Response.StatusCode != 200 {
        errorText := fmt.Sprintf("Expecting 200, got %d\n",
            client.Response.StatusCode)
        fmt.Println(errorText)
    }
}

output will be:

Request url: http://stackoverflow.com/favicon.ico/

Expecting 200, got 404

Can you fix that?

Best regards,
Dobrosław Żybort

It should be possible to provide the payload as a struct

I often find myself submitting JSON payloads to REST API's, but this library currently only allows for payloads in the form of url.Values.

Are PR's welcome?

Add method SetBasicAuth for rest.Client

Similar to http://golang.org/src/pkg/net/http/request.go?s=16438:16495#L507

Parse json arrays as go slices

The response is only unmarshaled if kind is a struct or a map. Why not a slice?

If you're open to a PR, I'd be happy to submit one with a test?

Add info to doc that client.To is needed if we want set custom url.Values

Sample:

client := rest.New("http://example.com/")

and now

client.Get(url.Values{
    "page": {"4"},
}).Do()

Will get http://example.com/

client.To("/").Get(url.Values{
    "page": {"4"},
}).Do()

Will get http://example.com/?page=4

Best regards,
Dobrosław Żybort