How to parse http Get request body in a Go http server?

Question

In a Go http server, I can get POST request body. Go net/http package seems to remove GET request body. I know it is better not to use http GET with request body,but I need to handle http GET with request body. Is it possible without changing the standard lib? Please help since I don't want to switch back to C with libevent!

When a client sends a POST with request body, below code will show the body content. But when a client sends a GET with request body, there is nothing in the body.

func handler(w http.ResponseWriter, r *http.Request) {
    defer r.Body.Close()
    body, _ := ioutil.ReadAll(r.Body)

    log.Printf("body: %v", string(body))
}

func main() {
    http.HandleFunc("/", handler)
    http.ListenAndServe(":8080", nil)
}

You "need to handle http GET with request body" ? Really ? There's probably a problem in your design. — Denys Séguret
– Denys Séguret, Commented Jul 16, 2013 at 12:21
Can you post an example of the code you want to implement? It's hard to understand what you're asking. — elithrar
– elithrar, Commented Jul 16, 2013 at 13:19
I took an admittedly quick look at Go's net/http source code, and I didn't find something like "drop the body if this is a GET". There are the expected conditions for server responses, like if method is a HEAD or if the status code is < 200, 204 or 304, I haven't seen nothing for client requests. Are you absolutely sure the client sends the body? Like checking the Request with Fiddler or something? — mna
– mna, Commented Jul 16, 2013 at 14:31
Somebody should edit the library so it's compliant, even if it's an uncommon use case. — joshlf
– joshlf, Commented Jul 16, 2013 at 15:38

Dave Grijalva · Accepted Answer · 2013-07-16 21:27:19Z

4

Most of the magic happens in transfer.go. Here's what I found that looks relevant in the fixLength func:

if !isResponse && requestMethod == "GET" {
    // RFC 2616 doesn't explicitly permit nor forbid an
    // entity-body on a GET request so we permit one if
    // declared, but we default to 0 here (not -1 below)
    // if there's no mention of a body.
    return 0, nil
}

Looks like, as long as your client is ending a Content-Length header, you're all good. If not, the library will assume there's no body on a GET request.

You're kind of off the edge of the map as your client is doing some pretty unusual/broken stuff. If you can fix the client, that's your best bet.

That said, if you have a client you need to support that's doing this wrong, you're going to have to roll some things yourself. You don't need to go all the way down to C and libevent. Simply copy the net/http package from the standard library into your project and modify it. Then change your import statements to point at your version of the library.

Alternatively, if you know that the client is not using keep-alive, you can Hijack the connection and just read whatever's left on the socket.

answered Jul 16, 2013 at 21:27

Dave Grijalva

1063 bronze badges

Sign up to request clarification or add additional context in comments.

1 Comment

gosharplite Over a year ago

You're correct. When GET does not specify Content-Length, r.Body is an eofReader (eofReader is a non-nil io.ReadCloser that always returns EOF).

Collectives™ on Stack Overflow

How to parse http Get request body in a Go http server?

1 Answer 1

1 Comment

Your Answer

Linked

Hot Network Questions

Collectives™ on Stack Overflow

1 Answer 1

1 Comment

Your Answer

Sign up or log in

Post as a guest

Linked

Related