Re: [PATCH] FEATURE/MAJOR: Add upstream-proxy-tunnel feature

2024-09-24 Thread Aleksandar Lazic
Hi Christopher. Thank you for your detail review. I will take some time for me to make the suggested changes. Will come back to the topic as soon as I find again time to dig into it. Best regards Alex On 2024-09-24 (Di.) 15:25, Christopher Faulet wrote: Le 19/09/2024 à 13:04, Aleksandar La

Re: [PATCH] FEATURE/MAJOR: Add upstream-proxy-tunnel feature

2024-09-24 Thread Christopher Faulet
Le 19/09/2024 à 13:04, Aleksandar Lazic a écrit : > Hi Willy. > > any chance to look into the Answer? > Hi Alex, I've started the review. First of all I agree with Willy. I guess it is not what many people have in mind when they are talking about supporting configuration of an HTTP proxy to c

Re: [PATCH] FEATURE/MAJOR: Add upstream-proxy-tunnel feature

2024-09-19 Thread Aleksandar Lazic
Hi Christopher. On 2024-09-19 (Do.) 16:00, Christopher Faulet wrote: Le 19/09/2024 à 13:04, Aleksandar Lazic a écrit : Hi Willy. any chance to look into the Answer? Hi Alex, In fact, Willy asked me to take a look on this feature. I started to work on it and I was distracted by the stable

Re: [PATCH] FEATURE/MAJOR: Add upstream-proxy-tunnel feature

2024-09-19 Thread Christopher Faulet
Le 19/09/2024 à 13:04, Aleksandar Lazic a écrit : Hi Willy. any chance to look into the Answer? Hi Alex, In fact, Willy asked me to take a look on this feature. I started to work on it and I was distracted by the stable releases and also by some tricky bugs. I'm going to start again to rev

Re: [PATCH] FEATURE/MAJOR: Add upstream-proxy-tunnel feature

2024-09-19 Thread Aleksandar Lazic
Hi Willy. any chance to look into the Answer? Regards Alex On 2024-08-15 (Do.) 10:55, Aleksandar Lazic wrote: Hi Willy. On 2024-08-12 (Mo.) 16:49, Willy Tarreau wrote: Hi Alex, On Mon, Aug 12, 2024 at 11:46:37AM +0200, Aleksandar Lazic wrote: On Thu, Jun 13, 2024 at 03:00:59AM +0200, Alek

Re: [PATCH] FEATURE/MAJOR: Add upstream-proxy-tunnel feature

2024-08-15 Thread Aleksandar Lazic
Hi Willy. On 2024-08-12 (Mo.) 16:49, Willy Tarreau wrote: Hi Alex, On Mon, Aug 12, 2024 at 11:46:37AM +0200, Aleksandar Lazic wrote: On Thu, Jun 13, 2024 at 03:00:59AM +0200, Aleksandar Lazic wrote: The final idea is something like this. ``` tcp-request content upstream-proxy-header Host %[r

Re: [PATCH] FEATURE/MAJOR: Add upstream-proxy-tunnel feature

2024-08-12 Thread Willy Tarreau
Hi Alex, On Mon, Aug 12, 2024 at 11:46:37AM +0200, Aleksandar Lazic wrote: > > On Thu, Jun 13, 2024 at 03:00:59AM +0200, Aleksandar Lazic wrote: > > > > The final idea is something like this. > > > > > > > > ``` > > > > tcp-request content upstream-proxy-header Host %[req.ssl_sni,lower] > > > > t

Re: [PATCH] FEATURE/MAJOR: Add upstream-proxy-tunnel feature

2024-08-12 Thread Aleksandar Lazic
Hi Willy. On 2024-08-12 (Mo.) 10:01, Willy Tarreau wrote: Hi Alex, I finally found time to have a look into this! Great :-) On Thu, Jun 13, 2024 at 03:00:59AM +0200, Aleksandar Lazic wrote: The final idea is something like this. ``` tcp-request content upstream-proxy-header Host %[req.ssl

Re: [PATCH] FEATURE/MAJOR: Add upstream-proxy-tunnel feature

2024-08-12 Thread Willy Tarreau
Hi Alex, I finally found time to have a look into this! On Thu, Jun 13, 2024 at 03:00:59AM +0200, Aleksandar Lazic wrote: > > The final idea is something like this. > > > > ``` > > tcp-request content upstream-proxy-header Host %[req.ssl_sni,lower] > > tcp-request content upstream-proxy-header "

Re: [PATCH] FEATURE/MAJOR: Add upstream-proxy-tunnel feature

2024-06-17 Thread Aleksandar Lazic
Hi. Updated patch. Changes: Set the right 'X' for upstream-proxy-header removed the upstream-proxy.png from patch git-format against latest master Any feedback and help is really appreciated. Best regards Alex On 2024-06-13 (Do.) 03:00, Aleksandar Lazic wrote: Hi. New Version. Changes: I

Re: [PATCH] FEATURE/MAJOR: Add upstream-proxy-tunnel feature

2024-06-12 Thread Aleksandar Lazic
Hi. Thanks for testing and feedback. On 2024-06-12 (Mi.) 20:35, Dave Cottlehuber wrote: On Wed, 12 Jun 2024, at 13:04, Aleksandar Lazic wrote: Hi. Attached a new version with updated upstream-proxy.cfg. This Patch have also the feature `upstream-proxy-target` to get rid of the dependency for

Re: [PATCH] FEATURE/MAJOR: Add upstream-proxy-tunnel feature

2024-06-12 Thread Dave Cottlehuber
On Wed, 12 Jun 2024, at 13:04, Aleksandar Lazic wrote: > Hi. > > Attached a new version with updated upstream-proxy.cfg. > > This Patch have also the feature `upstream-proxy-target` to get rid of the > dependency for the srv->hostname. > > ``` > tcp-request content upstream-proxy-target www.test1.

Re: [PATCH] FEATURE/MAJOR: Add upstream-proxy-tunnel feature

2024-06-12 Thread Aleksandar Lazic
Hi. Attached a new version with updated upstream-proxy.cfg. This Patch have also the feature `upstream-proxy-target` to get rid of the dependency for the srv->hostname. ``` tcp-request content upstream-proxy-target www.test1.com ``` Now have I tested the setup with `0.0.0.0` as server. ```

Re: [PATCH] FEATURE/MAJOR: Add upstream-proxy-tunnel feature

2024-06-12 Thread Aleksandar Lazic
Hi Dave. On 2024-06-12 (Mi.) 12:45, Aleksandar Lazic wrote: On 2024-06-12 (Mi.) 12:26, Dave Cottlehuber wrote: On Tue, 11 Jun 2024, at 22:57, Aleksandar Lazic wrote: Hi Dave. Thank you for your test and feedback. When you put this line into backend, will this be better? ``` tcp-request co

Re: [PATCH] FEATURE/MAJOR: Add upstream-proxy-tunnel feature

2024-06-12 Thread Aleksandar Lazic
On 2024-06-12 (Mi.) 12:26, Dave Cottlehuber wrote: On Tue, 11 Jun 2024, at 22:57, Aleksandar Lazic wrote: Hi Dave. Thank you for your test and feedback. When you put this line into backend, will this be better? ``` tcp-request connection upstream-proxy-header HOST www.httpbun.com ``` Rega

Re: [PATCH] FEATURE/MAJOR: Add upstream-proxy-tunnel feature

2024-06-12 Thread Dave Cottlehuber
On Tue, 11 Jun 2024, at 22:57, Aleksandar Lazic wrote: > Hi Dave. > > Thank you for your test and feedback. > > When you put this line into backend, will this be better? > > ``` > tcp-request connection upstream-proxy-header HOST www.httpbun.com > ``` > > Regards > Alex Hi Alex, Sorry I forgot to

Re: [PATCH] FEATURE/MAJOR: Add upstream-proxy-tunnel feature

2024-06-11 Thread Aleksandar Lazic
Hi Dave. Thank you for your test and feedback. When you put this line into backend, will this be better? ``` tcp-request connection upstream-proxy-header HOST www.httpbun.com ``` Regards Alex On 2024-06-11 (Di.) 23:52, Dave Cottlehuber wrote: On Mon, 10 Jun 2024, at 22:09, Aleksandar Lazic w

Re: [PATCH] FEATURE/MAJOR: Add upstream-proxy-tunnel feature

2024-06-11 Thread Dave Cottlehuber
On Mon, 10 Jun 2024, at 22:09, Aleksandar Lazic wrote: > It is now possible to set via "tcp-request connection upstream-proxy-header" > headers for the upstream proxy > > ``` > tcp-request connection upstream-proxy-header Host www.test1.com > tcp-request connection upstream-proxy-header Proxy-Auth