You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@trafficserver.apache.org by zw...@apache.org on 2014/01/21 00:08:19 UTC
[40/50] git commit: TS-2499: header_rewrite: fixed value substitution
TS-2499: header_rewrite: fixed value substitution
This commit fixes substitution of following variables in add header:
%<cquup>
%<cqhm>
%<cqhl>
%<chi>
%<port>
%<proto>
Pointy hat to: me (rbtz@)
Project: http://git-wip-us.apache.org/repos/asf/trafficserver/repo
Commit: http://git-wip-us.apache.org/repos/asf/trafficserver/commit/db5e07c4
Tree: http://git-wip-us.apache.org/repos/asf/trafficserver/tree/db5e07c4
Diff: http://git-wip-us.apache.org/repos/asf/trafficserver/diff/db5e07c4
Branch: refs/heads/5.0.x
Commit: db5e07c485c4a495c5a6e0ed5ec427236da3597e
Parents: 9ca808b
Author: Alexey Ivanov <ai...@linkedin.com>
Authored: Wed Jan 15 18:14:05 2014 -0800
Committer: Alexey Ivanov <ai...@linkedin.com>
Committed: Wed Jan 15 21:20:15 2014 -0800
----------------------------------------------------------------------
plugins/header_rewrite/operators.cc | 3 +--
1 file changed, 1 insertion(+), 2 deletions(-)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/trafficserver/blob/db5e07c4/plugins/header_rewrite/operators.cc
----------------------------------------------------------------------
diff --git a/plugins/header_rewrite/operators.cc b/plugins/header_rewrite/operators.cc
index fda921c..da14d35 100644
--- a/plugins/header_rewrite/operators.cc
+++ b/plugins/header_rewrite/operators.cc
@@ -383,12 +383,11 @@ void
OperatorAddHeader::exec(const Resources& res) const
{
std::string value;
+ _value.append_value(value, res);
VariableExpander ve(value);
value = ve.expand(res);
- _value.append_value(value, res);
-
// Never set an empty header (I don't think that ever makes sense?)
if (value.empty()) {
TSDebug(PLUGIN_NAME, "Would set header %s to an empty value, skipping", _header.c_str());