Github user shinrich commented on the pull request: https://github.com/apache/trafficserver/pull/350#issuecomment-159436716 W.r.t. UnixNetVConnection pulling in Diags, that is kind of disturbing and can be cleaned up I think. The network area can read in the client_ip config and do the test. If we decide to keep the config-based control, I'll fix that up. Of course we are dragging in the global diag everywhere we are calling the Debug() macros. I removed the extra reference to Diags.h, and everything still built.
--- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---