Re: A patch for haproxy-1.3.17 which add X-Original-Dst header.

Hi Maik,

On Fri, May 08, 2009 at 04:44:05PM +0200, Maik Broemme wrote:
> Hi,
>
> many thanks Willy, I have still found a line which was missing in the
> patch. In 'src/cfgparse.c' around line 677 there are the default values
> set. I missed the following line:
>
> curproxy->except_to = defproxy.except_to;
>
> Just to set the default values if they will be - maybe - changed in
> future versions. So please add it before adding the patch. :)

OK thanks for this one. I tend to forget those too :-)

> I have also another question: I am preparing a patch for setting
> multiple except values to X-Forward-For and X-Original-To, because I am
> still missing this feature. Am I right that it isn't possible in the
> current version or did i miss something in documentation?

indeed, you can only set one exception. But I'd rather use ACLs to add such headers. That way, it will be possible to set them based on a lot of other conditions, not just source address.

Regards,
Willy Received on 2009/05/08 16:58