Hi Jeremie,
Jeremie Le Hen wrote:
Hi Bogdan-Andrei,
thank your for your quick and clear answer.
On Tue, Sep 26, 2006 at 11:05:09PM +0300, Bogdan-Andrei Iancu wrote:
append_hf() adds an hdr to a request and not to a
reply - see docs
(textops module). For you purpose, use append_to_reply() in failure_route.
I admit I have neglected to look closely at the documentation, but
in my humble opinion, the name is misleading. It sounds as a general
way to append a header field, without suggesting it only work for
a request. I guess there is an historical reason for this,
yes, that is right
but
what is the technival reason for not making it work for the replies ?
you need separate functions for adding to request and adding to reply.
For example you process a request and you want to add a header to 1) the
current processed request or 2) to the reply that will be sent for this
request.
Furthermore, the append_hf() function is described as
being usable
in REQUEST_ROUTE, ONREPLY_ROUTE, FAILURE_ROUTE and BRANCH_ROUTE ?
If it does naught on replies, what's the point of using it in
ONREPLY_ROUTE and FAILURE_ROUTE ?
well...it is a bit complicate to explain here....is not quite a doc
error: actually append_hf() adds a header to the current process message
which can be
1) request in REQUEST_ROUTE, FAILURE_ROUTE and BRANCH_ROUTE
2) reply in ONREPLY_ROUTE
append_to_reply() will set the header to be appended to all replies
generated locally.
regards,
bogdan