Re: Add 401, 403 retries at l7

2020-11-23 Thread Christopher Faulet
Le 22/11/2020 à 15:40, Julien Pivotto a écrit : :facepalm: Here is the good one Now merged. Thanks Julien ! (Note: I added the "stream" label to the commit message) -- Christopher Faulet

Re: Add 401, 403 retries at l7

2020-11-22 Thread Julien Pivotto
:facepalm: Here is the good one On 22 Nov 15:39, Tim Düsterhus wrote: > Julien, > > Am 22.11.20 um 15:24 schrieb Julien Pivotto: > > Here you go. > > > > It looks like you sent the same patch again. > > Best regards > Tim Düsterhus -- (o-Julien Pivotto //\Open-Source Consultant

Re: Add 401, 403 retries at l7

2020-11-22 Thread Tim Düsterhus
Julien, Am 22.11.20 um 15:24 schrieb Julien Pivotto: > Here you go. > It looks like you sent the same patch again. Best regards Tim Düsterhus

Re: Add 401, 403 retries at l7

2020-11-22 Thread Julien Pivotto
Here you go. On 20 Nov 12:51, Christopher Faulet wrote: > Le 12/11/2020 à 11:18, Julien Pivotto a écrit : > > Dear, > > > > Please find a patch to add 401 and 403 l7 retries, see > > https://github.com/haproxy/haproxy/issues/948 > > > > Thanks Julien. So

Re: Add 401, 403 retries at l7

2020-11-20 Thread Willy Tarreau
On Fri, Nov 13, 2020 at 01:30:21PM +0100, Julien Pivotto wrote: > On 13 Nov 00:12, Jonathan Matthews wrote: > > I think retry on 4XX, without modifying the request, is a terrible idea. > > It's pretty much the opposite of what the HTTP spec says, and isn't > > something haproxy should learn how to

Re: Add 401, 403 retries at l7

2020-11-20 Thread Christopher Faulet
Le 12/11/2020 à 11:18, Julien Pivotto a écrit : Dear, Please find a patch to add 401 and 403 l7 retries, see https://github.com/haproxy/haproxy/issues/948 Thanks Julien. Sorry for the delay. Sure we can add it. However could you adapt your commit message to follow CONTRIBUTING advises

Re: Add 401, 403 retries at l7

2020-11-13 Thread Julien Pivotto
On 13 Nov 00:12, Jonathan Matthews wrote: > On Thu, 12 Nov 2020 at 12:21, Julien Pivotto wrote: > > > Dear, > > > > Please find a patch to add 401 and 403 l7 retries, see > > https://github.com/haproxy/haproxy/issues/948 > > > Hey Julien, >

Re: Add 401, 403 retries at l7

2020-11-12 Thread Jonathan Matthews
On Thu, 12 Nov 2020 at 12:21, Julien Pivotto wrote: > Dear, > > Please find a patch to add 401 and 403 l7 retries, see > https://github.com/haproxy/haproxy/issues/948 Hey Julien, This really feels like an anti-feature, to be frank! If a specific backend server can’t auth anyone

Add 401, 403 retries at l7

2020-11-12 Thread Julien Pivotto
Dear, Please find a patch to add 401 and 403 l7 retries, see https://github.com/haproxy/haproxy/issues/948 -- (o-Julien Pivotto //\Open-Source Consultant V_/_ Inuits - https://www.inuits.eu From f71e0b2eb69303fa59645fefda3960fb2a9eb7fb Mon Sep 17 00:00:00 2001 From: Julien Pivotto