Commit b72325fd authored by Brad Fitzpatrick's avatar Brad Fitzpatrick

net/http: remove some obsolete TODOs

We've decided to leave logging to third-parties (there are too
many formats), which others have done.

And we can't change the behavior of the various response
fields at this point anyway. Plus I argue they're correct and
match their documention.

R=golang-dev, r
CC=golang-dev
https://golang.org/cl/8391043
parent d97157d3
......@@ -112,8 +112,8 @@ var respTests = []respTest{
ProtoMinor: 0,
Request: dummyReq("GET"),
Header: Header{
"Connection": {"close"}, // TODO(rsc): Delete?
"Content-Length": {"10"}, // TODO(rsc): Delete?
"Connection": {"close"},
"Content-Length": {"10"},
},
Close: true,
ContentLength: 10,
......@@ -170,7 +170,7 @@ var respTests = []respTest{
Request: dummyReq("GET"),
Header: Header{},
Close: false,
ContentLength: -1, // TODO(rsc): Fix?
ContentLength: -1,
TransferEncoding: []string{"chunked"},
},
......
......@@ -4,9 +4,6 @@
// HTTP server. See RFC 2616.
// TODO(rsc):
// logging
package http
import (
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment