Why http/1.0 is forced?

[23doors]

Any particular reason why http/1.0 is forced here:
https://github.com/mattn/go-uwsgi/blob/master/uwsgi.go#L251 and here https://github.com/mattn/go-uwsgi/blob/master/uwsgi.go#L273 ?

[mattn]

I'm not sure, I don't remember, but maybe, when I tested this, the server did not return a response. This is workaround.

[23doors]

Thanks for quick answer!
Tried removing these 2 lines and everything seemed to be working fine at least for http/1.1 but I haven't done any extensive tests. And I am running it behind a uwsgi (internal routing) which may for some reason yield different results than running it behind nginx + uwsgi pass.

[mattn]

If you solve your issue, please send me p-r.

[23doors]

#5

[23doors]

Any chance you could take a look? If you don't have time right now, I'll just use my fork for the time being

[mattn]

Test fails. Could you handle this? https://travis-ci.org/mattn/go-uwsgi

[23doors]

Made a fix for regression: #6 but actually it seems that tests were failing for a long time now. Fixed the error, but tests seem to hang anyway. I tested version before PR - and it also hangs on me so it doesn't seem to be a regression. Sadly, I don't have time right now to debug what is wrong with the testcase especially as it looks unfinished.

Anyway. TestBasic should work after this PR.
TestServer is another story though.
From what I gathered:

passenger := &Passenger{"tcp", "127.0.0.1"}

this is wrong, no port in addr - and server that it points to is not running

l, err := net.Listen("tcp", "127.0.0.1:0")

listener is declared but unused. And yet we later on try to connect to it which causes timeout.

[mattn]

I removed TestServer now because I don't remember what I want to test.

[mattn]

closed by #8