Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

net/http/cgi: REQUEST_URI is not available on IIS #4367

Closed
gopherbot opened this issue Nov 8, 2012 · 7 comments
Closed

net/http/cgi: REQUEST_URI is not available on IIS #4367

gopherbot opened this issue Nov 8, 2012 · 7 comments
Labels
FrozenDueToAge
Milestone
Go1.1

Comments

@gopherbot
Copy link

by mikael.miffe.eriksson:

What steps will reproduce the problem?
1. Using http://play.golang.org/p/HW0nuV59QY as a CGI app on IIS
2. Visiting http://the-cgi-url/HelloWorld in a webbrowser

What is the expected output?
URL: /HelloWorld

What do you see instead?
URL: 

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

Which operating system are you using?
Windows 7 32bit

Which version are you using?  (run 'go version')
go version go1.0.3

Please provide any additional information below.
The cgi package uses the REQUEST_URI to get the path/query information of the request,
but REQUEST_URI is not available in IIS. REQUEST_URI is not mentioned in the CGI spec
either. The same information can be assembled from PATH_INFO, SCRIPT_NAME and
QUERY_STRING.
@alexbrainman
Copy link
Member

Comment 1:

I know nothing about cgi, so I could be wrong here. But, searching in
http://tools.ietf.org/html/rfc3875, I find no reference to REQUEST_URI. So, perhaps, you
are correct in your assessment.
I will accept the issue as valid, assuming others do not object.
If you have a fix, feel free to send it as per http://golang.org/doc/contribute.html.
Thank you.
Alex

Labels changed: added packagebug.

Status changed to Accepted.

@bradfitz
Copy link
Contributor

Comment 2:

I can get to this eventually if nobody else does, but Mikael fixing it would be best,
since he can actually verify the fix with IIS, which I don't have.

@rsc
Copy link
Contributor

rsc commented Dec 10, 2012

Comment 3:

Labels changed: added size-m.

@minux
Copy link
Member

minux commented Dec 20, 2012

Comment 4:

Issue #4575 has been merged into this issue.

@rsc
Copy link
Contributor

rsc commented Dec 30, 2012

Comment 5:

Labels changed: added priority-later, removed priority-triage.

@alexbrainman
Copy link
Member

Comment 6:

https://golang.org/cl/7062052/

Status changed to Started.

@alexbrainman
Copy link
Member

Comment 7:

This issue was closed by revision 548e587.

Status changed to Fixed.

@minux minux mentioned this issue Dec 8, 2014
Closed
@rsc rsc added this to the Go1.1 milestone Apr 14, 2015
@rsc rsc removed the go1.1 label Apr 14, 2015
@golang golang locked and limited conversation to collaborators Jun 24, 2016
This issue was closed.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
FrozenDueToAge
Projects
None yet
Milestone
Go1.1
Development

No branches or pull requests
5 participants
@bradfitz @rsc @minux @gopherbot @alexbrainman