Fix broken wsgi.py due to sockname info size

[merrellb]

Addresses issue introduced by cccb9cb

[merrellb]

This failed immediately on any request when using aiobottle. Not sure how their codepath differs from existing wsgi tests.

[redixin]

getsockname may return more then 2 values, e.g. in case of AF_INET6 (host, port, flowinfo, scopeid)

Probably we should check the address family and call appropriate code to fill in variables.

[merrellb]

On my OSX box I do get back the four part tuple. I can't say I am familiar with flowinfo or scopeid so it isn't clear how helpful they would be to detect and add them to the 'environ' variable. Given the move to tuple unpacking in the earlier commit, this seemed like the quickest way to get wsgi working again :-)

[asvetlov]

I'm with @redixin
We should explicitly process 3 address families: AF_INET, AF_INET6 and AF_UNIX without relying on presence of peername or length of address tuple.

[redixin]

@asvetlov I've made a patch, but it adds one extra call get_extra_info('socket') to query address family. The problem is this will be called at every request, but may be called once at start of the daemon. Could you please give me a glue how this can be done?

[asvetlov]

get_extra_info(socket) is a dictionary lookup, not a syscall.
socket.family is not a syscall also.
Feel free to get the info on every request.

[redixin]

@asvetlov @merrellb please take a look #718

[asvetlov]

Fixed by #718

[lock]

This thread has been automatically locked since there has not been
any recent activity after it was closed. Please open a new issue for
related bugs.

If you feel like there's important points made in this discussion,
please include those exceprts into that new issue.