On Wed, Jan 14, 2009 at 7:25 AM, Jeff King <peff@xxxxxxxx> wrote: > So at the very least, you should be adding REMOTE_HOST in _addition_ to > REMOTE_ADDR, not instead of. But that still leaves one final concern, > which is that some git-daemon admins might not want to pay the cost for > a reverse lookup for every request. It's extra network traffic, and adds > extra latency to the process (but I don't personally run git-daemon, and > I don't know whether big sites like kernel.org actually care about > this). Speaking for large sites everywhere, yes they do care. Enabling DNS lookups must be configurable. j. -- To unsubscribe from this list: send the line "unsubscribe git" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html