golang / go

The Go programming language
https://go.dev
BSD 3-Clause "New" or "Revised" License
122.98k stars 17.53k forks source link

net/http/cgi: No way to pass error io.Writer to CGI (including patch) #7197

Closed gopherbot closed 8 years ago

gopherbot commented 10 years ago

by steven.hartland@multiplay.co.uk:

What steps will reproduce the problem?
If possible, include a link to a program on play.golang.org.
1. Use CGI and attempt to redirect STDERR to an alternative location

What is the expected output?
The ability to configure the error io.Writer should be present

What do you see instead?
The ability to configure the error io.Writer is not present

Which compiler are you using (5g, 6g, 8g, gccgo)?
6g

Which operating system are you using?
FreeBSD 8.3-RELEASE

Which version are you using?  (run 'go version')
go version go1.2 freebsd/amd64

Please provide any additional information below.
Apply the attached patch which adds the Stderr option

Attachments:

  1. cgi-err.patch (1025 bytes)
bradfitz commented 10 years ago

Comment 1:

We don't attach patches to the bug tracker.  Our development and code review workflow is
documented at:
http://golang.org/doc/contribute.html
Be sure to submit a CLA as well:
http://golang.org/doc/contribute.html#copyright
davecheney commented 10 years ago

Comment 2:

Please submit your patch via the CLA process outlined in comment number 1.

Status changed to WaitingForReply.

rsc commented 10 years ago

Comment 3:

Labels changed: added repo-main, release-none.

bradfitz commented 8 years ago

@stevenh has signed the CLA and submitted stuff in the past, so I'll send his patch out in Gerrit: https://go-review.googlesource.com/21196

gopherbot commented 8 years ago

CL https://golang.org/cl/21196 mentions this issue.