[GitHub] trafficserver issue #588: TS-4291 Adds log field "pqnhl" which ignores inter...

2016-12-16 Thread ogoodman
Github user ogoodman commented on the issue: https://github.com/apache/trafficserver/pull/588 Hi Bryan, Sorry, I probably should have let you know sooner. Please feel free to close any pull requests of mine. I have changed jobs and no longer work with Traffic

[GitHub] trafficserver issue #588: TS-4291 Adds log field "pqnhl" which ignores inter...

2016-12-16 Thread bryancall
Github user bryancall commented on the issue: https://github.com/apache/trafficserver/pull/588 @ogoodman We talked about this in the github PR meeting and would like to remove logging of the @ headers for the current tags. --- If your project is set up for it, you can reply to

[GitHub] trafficserver issue #588: TS-4291 Adds log field "pqnhl" which ignores inter...

2016-08-20 Thread jpeach
Github user jpeach commented on the issue: https://github.com/apache/trafficserver/pull/588 @zwoop @ogoodman I agree that the logging tags should ignore any headers not send on the wire. Let's fix the existing tags rather than make new ones. I would say the current behavior is just a

[GitHub] trafficserver issue #588: TS-4291 Adds log field "pqnhl" which ignores inter...

2016-07-20 Thread zwoop
Github user zwoop commented on the issue: https://github.com/apache/trafficserver/pull/588 So, now that we're on the 7.0.0 release cycle, I wonder if we should have pqhl, cqhl, pshl, and sshl all exclude any @ headers? If not, if we really want this, don't we also want all 4 variants