On Thu, 17 Sep 2009, Willy Tarreau wrote:

Hi Krzysztof,

On Thu, Sep 17, 2009 at 04:21:20PM +0200, Krzysztof Oledzki wrote:
(...)
Strange, I must have overlooked something, probably because I don't use
pure tcp mode so my tests were rather limited, even too limited. :(

No problem, what matters is that it's improving :-)

Yep, silly overlook:

--- a/src/checks.c
+++ b/src/checks.c
@@ -523,6 +523,7 @@ static int event_srv_chk_w(int fd)

                        /* good TCP connection is enough */
                        s->result |= SRV_CHK_RUNNING;
+                       set_server_check_status(s, HCHK_STATUS_L4OK);
                        goto out_wakeup;
                }
        }


I'll do some more tests to make sure all cases are now really covered and send the final patch, I hope. ;) Then we may talk about more improvements. :)

Thanks,

                        Krzysztof Olędzki

Reply via email to