Updated Branches: refs/heads/master 9ca808b67 -> 7c5755105
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/master Commit: db5e07c485c4a495c5a6e0ed5ec427236da3597e Parents: 9ca808b Author: Alexey Ivanov <aiva...@linkedin.com> Authored: Wed Jan 15 18:14:05 2014 -0800 Committer: Alexey Ivanov <aiva...@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());