net/http/cgi: support Python etc non-executable scripts on Windows

[zhw2101024]

Please answer these questions before submitting your issue. Thanks!

What version of Go are you using (go version)?

1.7.4

What operating system and processor architecture are you using (go env)?

set GOARCH=amd64
set GOBIN=
set GOEXE=.exe
set GOHOSTARCH=amd64
set GOHOSTOS=windows
set GOOS=windows
set GOPATH=
set GORACE=
set GOROOT=D:\Go
set GOTOOLDIR=D:\Go\pkg\tool\windows_amd64
set CC=gcc
set GOGCCFLAGS=-m64 -mthreads -fmessage-length=0
set CXX=g++
set CGO_ENABLED=1

What did you do?

If possible, provide a recipe for reproducing the error.
A complete runnable program is good.
A link on play.golang.org is best.

cgidemo.zip
Change to unziped directory, and execute go run cgi.go, then visit http://localhost:8000/cgi-bin/test.py

What did you expect to see?

Hello on my browser

What did you see instead?

The page hangs up

[robothor]

This works for me if I make sure test.py is executable:

#!/usr/bin/env python

print("Content-Type: text/html\n")
print("Hello")

Also, there were some strange whitespace characters in the file that were causing problems.

Note that I am running on darwin and am not really sure how to fix this on Windows.

[minux]

You need to somehow make the program test.py runnable on Windows. On Unix, you can add a shebang header and make the script executable. I don't know how to achieve that on windows. I don't think this is a Go bug. Please ask questions on other mediums, see https://golang.org/wiki/Questions

[bradfitz]

@minux, but "runnable" on a file is a concept that doesn't exist on Windows.

We should do whatever other popular CGI servers do. (Whatever that is.) Shebang support, perhaps?

[minux]

On windows, it's likely based on file suffix alone. @alexbrainmain. One simple solution is perhaps using "cmd /c start script.py arguments...", but I don't know the implications.

[mattn]

For Windows, you should handle extension whether it will be runnable.

https://github.com/mattn/go-cgiserver/blob/master/cgiserver.go

BTW: s/alexbrainmain/alexbrainman/

[zhw2101024]

There's a WINAPI called ShellExecute, equals to cmd /c script.py, but have no idea if it's applicable to use this here. Maybe will link to windows dll file.
cmd should work, as all supported windows versions have cmd and cmd's behavior seems won't change.

[zhw2101024]

@bradfitz
Thanks for reopening this issue.
Please allow me to clarify this issue with more details.
currently golang's cgi only supports .com, .exe, .bat and .cmd files. If one want to use golang's cgi on windows, there's only two choices: one of compiling languages, or bat script. compiling languages have many advantages, but not so handy as script languages such as perl, one of most popular cgi backend languages. On the other hand, bat lack too many modern features, so it's difficult to implement a complete service with just bat.
So if cgi support script languages, it will be far more useful than before.

[minux]

You just need to use a special shebang line for Python scripts on Windows (and also save the script with suffix .bat): @"c:\python27\python.exe" -x %~f0 & exit /b I think this is also possible for Perl (it's probably easier for Perl, because perl's syntax is not as strict as Python's, and I have to resort to use -x to write a Python/bat polyglot.) While Windows batch files are quite limiting compared to Unix shell scripts, this is still possible.

[zhw2101024]

Thank you, This works for me at least.
@minux
But comparing with Python's cgi module or Apache's cgi module, this still have problems.
Imaging you have tones of python script use as cgi backend, and use apache's cgi module. One day you want to use golang's cgi, then you need to modify all existing scripts. A new batch file may do this work for you, but python scripts with bat suffix may terribly misleading your IDE or code editor, or other maintainers.