tree 205ec99ae852e2c2b52e6a266c8d56840d8b5c50
parent 46175dd81d06d733adbfd4b475bb36fa4273080d
author Willy Tarreau <w@1wt.eu> 1436002590 +0200
committer Willy Tarreau <w@1wt.eu> 1436002590 +0200
encoding latin1

BUG/MAJOR: tcp: tcp rulesets were still broken

Commit cc87a11 ("MEDIUM: tcp: add register keyword system.") broke the
TCP ruleset by merging custom rules and accept. It was fixed a first time
by commit e91ffd0 ("BUG/MAJOR: tcp: only call registered actions when
they're registered") but the accept action still didn't work anymore
and was causing the matching rule to simply be ignored.

Since the code introduced a very fragile behaviour by not even mentionning
that accept and custom were silently merged, let's fix this once for all by
adding an explicit check for the accept action. Nevertheless, as previously
mentionned, the action should be changed so that custom is the only action
and the continue vs break indication directly comes from the callee.

No backport is needed, this bug only affects 1.6-dev.