Matt Sergeant wrote: > > Why not leave it as NXDOMAIN or SERVFAIL or TIMEOUT in case of > errors? I find this useful in my Received headers...
I didn't know that you coded it that way on purpose, instead it looked to me like a thing that could be improved. I think that remote_host should not be the error code in case of errors. That contradicts the description of the method and may force other people using remote_host to do their own checks. What do you think about setting only remote_info to either the result of a successful lookup or the error code? And remote_host will be either the result of the successful lookup or "[$remote_ip]" in case of errors?
