net/http: remove logging on bad client requests

As discussed in #18095 the server should not log for bad user input.

Change-Id: I628a796926eff3a971e5b04abec17ea377c3f9b7
Reviewed-on: https://go-review.googlesource.com/33617
Reviewed-by: Brad Fitzpatrick <[email protected]>
Run-TryBot: Brad Fitzpatrick <[email protected]>
TryBot-Result: Gobot Gobot <[email protected]>

Author: kennygrant

src/net/http/server.go

@@ -1763,7 +1763,6 @@ func (c *conn) serve(ctx context.Context) {
 				// while they're still writing their
 				// request. Undefined behavior.
 				const publicErr = "431 Request Header Fields Too Large"
-				c.server.logf("http: %s", publicErr)
 				fmt.Fprintf(c.rwc, "HTTP/1.1 "+publicErr+errorHeaders+publicErr)
 				c.closeWriteAndWait()
 				return
@@ -1777,7 +1776,6 @@ func (c *conn) serve(ctx context.Context) {
 				publicErr = publicErr + ": " + string(v)
 			}
 
-			c.server.logf("http: %s", publicErr)
 			fmt.Fprintf(c.rwc, "HTTP/1.1 "+publicErr+errorHeaders+publicErr)
 			return
 		}