Skip to content

Commit 1d2557d

Browse files
committed
revert mistaken change to header rewrite set-header
1 parent 6e023c8 commit 1d2557d

File tree

1 file changed

+4
-3
lines changed

1 file changed

+4
-3
lines changed

plugins/header_rewrite/operators.cc

Lines changed: 4 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -741,6 +741,8 @@ void
741741
OperatorSetHeader::initialize(Parser &p)
742742
{
743743
OperatorHeaders::initialize(p);
744+
745+
_value.set_value(p.get_value(), this);
744746
}
745747

746748
bool
@@ -752,7 +754,7 @@ OperatorSetHeader::exec(const Resources &res) const
752754

753755
// Never set an empty header (I don't think that ever makes sense?)
754756
if (value.empty()) {
755-
Dbg(pi_dbg_ctl, "Empty value provided, clearning strategy");
757+
Dbg(pi_dbg_ctl, "Would set header %s to an empty value, skipping", _header.c_str());
756758
return true;
757759
}
758760

@@ -1591,13 +1593,12 @@ OperatorSetNextHopStrategy::initialize(Parser &p)
15911593
Operator::initialize(p);
15921594

15931595
_value.set_value(p.get_arg(), this);
1594-
Dbg(pi_dbg_ctl, "OperatorSetNextHopStrategy::initiazlize: %s", _value.get_value().c_str());
1596+
Dbg(pi_dbg_ctl, "OperatorSetNextHopStrategy::initialie: %s", _value.get_value().c_str());
15951597
}
15961598

15971599
void
15981600
OperatorSetNextHopStrategy::initialize_hooks()
15991601
{
1600-
// add_allowed_hook(TS_HTTP_PRE_REMAP_HOOK);
16011602
add_allowed_hook(TS_HTTP_READ_REQUEST_HDR_HOOK);
16021603
add_allowed_hook(TS_REMAP_PSEUDO_HOOK);
16031604
}

0 commit comments

Comments
 (0)