Status should always be initialised in rlm_ldap_bind

We'll never actually hit this (it would require fr_connection_get_num to return a negative value).
Don't set nonblock on outgoing client connections.

The caller will need to set it if necessary
Fix use of fr_nonblock && listen() for sockets

Outgoing TCP sockets are non-blocking.  Incoming sockets
are non-blocking.  Outgoing TLS sockets are blocking.

Don't call listen() on outgoing sockets
certs will be NULL for outgoing proxy sockets.
Add the proxy listener to the packet list immediately

so that we can allocate IDs immediately
only use TLS if TLS is enabled
close_notify isn't an error.  It's a polite notification
Don't print out useless proxy ID
It's OK to send packets in outgoing proxy socket INIT state
Formatting
Comment
Remove code in radius_map2vp for !* ANY (it's not needed)
Fix double & with attribute references in debug_map

It's completely pointless, so much so there's not
Print maps for !* ANY
Add macro for FR_FAULT_LOG
Use pairmake_packet for Module-Failure-Message
Move last few conf items to FR_CONF_OFFSET

and fix resulting compiler warnings
Don't return OK for EAP-MSCHAPv2 success/fail

it breaks other things in the server, when we have

	eap {
		ok = return
	}

There is minimal additional cost to doing this
Remove one last check for zero
Track response_window to request->delay.  Addresses #645

We track the min_response_window for all home servers, and
then use that value when setting request->delay.  This lets us
deal more quickly with home servers which need sub-second timers