server_ip is likely not defined correctly.

[MattMenke2]

The spec defines "server_ip" as "The IP address of the server to which the user agent sent the request, if available. Otherwise, an empty string." Yet in Example 10, server_ip is set in the case a request was not sent due to not being able to establish a connection. While examples are typically non-normative, I suspect in this case it's the example that's correct, and server_ip should not be empty even in the case no request was actually sent.

[MattMenke2]

A related issue - if we're unable to establish a TCP connection at all, then multiple IPs will be tried. If a hostname maps to 5 IP addresses, and they all fail, what should server_ip be? The first one we tried? The last one?