[
https://issues.apache.org/jira/browse/TS-4449?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15347063#comment-15347063
]
ASF GitHub Bot commented on TS-4449:
------------------------------------
Github user zwoop commented on a diff in the pull request:
https://github.com/apache/trafficserver/pull/735#discussion_r68303567
--- Diff: plugins/header_rewrite/conditions.cc ---
@@ -316,12 +314,12 @@ ConditionQuery::eval(const Resources &res)
{
std::string s;
+ TSDebug(PLUGIN_NAME, "Evaluating QUERY()");
if (NULL == res._rri) {
- TSDebug(PLUGIN_NAME, "QUERY requires remap initialization! Evaluating
to false!");
+ TSDebug(PLUGIN_NAME, "\tQUERY requires remap initialization!
Evaluating to false!");
--- End diff --
Maybe, but that might be a different bug to fix honestly. File a Jira? We
shouldn't error out here I don't think.
> header_rewrite: Improve TSDebug() statements
> --------------------------------------------
>
> Key: TS-4449
> URL: https://issues.apache.org/jira/browse/TS-4449
> Project: Traffic Server
> Issue Type: Improvement
> Components: Plugins
> Reporter: Leif Hedstrom
> Assignee: Leif Hedstrom
> Fix For: 7.0.0
>
>
> Right now, it can be difficult at times to understand why header_rewrite
> isn't behaving as you'd expect. There are a number of places where we can
> improve TSDebug().
> Also, I'm going to do a code cleanup here, to make the code more inline with
> our current best practices (e.g. use if () { } consistently).
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
