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.

This change is motivated by that fact that we use it for UNIX domain
socket as well as IPv4.

This commit also fixes environment variables used to tell inherited
file descriptors to new binary are stacked up each time new binary is
executed.

This is typically programming mistake that we did not check return
value before doing another.