Fixes GH-170

Fixes GH-172

declaration of make_socket_closeonexec need the proper scope here, it was there per request at https://github.com/tatsuhiro-t/nghttp2/pull/142 , not sure why is was removed.

This reverts commit ef090d42.

Merge branch 'bcard/address-nghttp2-server' of https://github.com/bcard/nghttp2 into bcard-bcard/address-nghttp2-server

Conflicts:
	src/nghttp.cc

This ensures that all frontend code calls
Downstream::end_upload_data() when request was all received.

to numeric_name(const struct sockaddr *sa, socklen_t salen) to remove
dependency on addrinfo struct.

This is a sign that we only use thread-local storage for logging only.

A part of an effort to eliminate thread_local WorkerConfig

Moving nghttp's numeric_name function to util.cc and using this to generate the address name in HttpServer.cc

This commit only affects the library behaviour unless
nghttp2_option_set_no_http_messaging() is used.

We like strict validation against header field name and value against
RFC 7230, but we have already so much web sites and libraries in
public internet which do not obey these rules.  Simply just
terminating stream because of this may break web sites and it is too
disruptive.  So we decided that we should be conservative here so
those header fields containing illegal characters are just ignored.
But we are conservative only for regular headers.  We are strict for
pseudo headers since it is new to HTTP/2 and new implementations
should know the rules better.