]> git.ipfire.org Git - thirdparty/haproxy.git/commitdiff
BUG/MINOR: stats: http-request rules still don't cope with stats
authorWilly Tarreau <w@1wt.eu>
Thu, 27 Dec 2012 09:34:21 +0000 (10:34 +0100)
committerWilly Tarreau <w@1wt.eu>
Thu, 27 Dec 2012 09:34:21 +0000 (10:34 +0100)
Since commit 20b0de5, we also had another remaining issue : an
"http-request allow" rule would prevent a stats rule from being
processed.

src/proto_http.c

index 535c159495ca913041316a98ad0ef80c23dcc555..3b5a478a204ba448d2f7eb9aa72aa96d82955efb 100644 (file)
@@ -3063,8 +3063,8 @@ int http_handle_stats(struct session *s, struct channel *req)
 
 /* Executes the http-request rules <rules> for session <s>, proxy <px> and
  * transaction <txn>. Returns NULL if it executed all rules, or a pointer to
- * the last rule if it had to stop before the end (auth, deny, allow). It may
- * set the TX_CLDENY on txn->flags if it encounters a deny rule.
+ * the last rule if it had to stop before the end (auth, deny). It may set
+ * the TX_CLDENY on txn->flags if it encounters a deny rule.
  */
 static struct http_req_rule *
 http_check_access_rule(struct proxy *px, struct list *rules, struct session *s, struct http_txn *txn)
@@ -3090,7 +3090,7 @@ http_check_access_rule(struct proxy *px, struct list *rules, struct session *s,
                if (ret) {
                        switch (rule->action) {
                        case HTTP_REQ_ACT_ALLOW:
-                               return rule;
+                               return NULL;
                        case HTTP_REQ_ACT_DENY:
                                txn->flags |= TX_CLDENY;
                                return rule;