Changed the order of handlers: protocol first, user second

cesanta · Jan 20, 2024 · c1f34e8 · c1f34e8
1 parent f883504
commit c1f34e8
Show file tree

Hide file tree

Showing 3 changed files with 5 additions and 9 deletions.
diff --git a/mongoose.c b/mongoose.c
@@ -1345,11 +1345,9 @@ void mg_call(struct mg_connection *c, int ev, void *ev_data) {
     MG_PROF_ADD(c, names[ev]);
   }
 #endif
-  // Run user-defined handler first, in order to give it an ability
-  // to intercept processing (e.g. clean input buffer) before the
-  // protocol handler kicks in
-  if (c->fn != NULL) c->fn(c, ev, ev_data);
+  // Fire protocol handler first, user handler second. See #2559
   if (c->pfn != NULL) c->pfn(c, ev, ev_data);
+  if (c->fn != NULL) c->fn(c, ev, ev_data);
 }
 
 void mg_error(struct mg_connection *c, const char *fmt, ...) {

diff --git a/src/event.c b/src/event.c
@@ -16,11 +16,9 @@ void mg_call(struct mg_connection *c, int ev, void *ev_data) {
     MG_PROF_ADD(c, names[ev]);
   }
 #endif
-  // Run user-defined handler first, in order to give it an ability
-  // to intercept processing (e.g. clean input buffer) before the
-  // protocol handler kicks in
-  if (c->fn != NULL) c->fn(c, ev, ev_data);
+  // Fire protocol handler first, user handler second. See #2559
   if (c->pfn != NULL) c->pfn(c, ev, ev_data);
+  if (c->fn != NULL) c->fn(c, ev, ev_data);
 }
 
 void mg_error(struct mg_connection *c, const char *fmt, ...) {

diff --git a/test/unit_test.c b/test/unit_test.c
@@ -1391,7 +1391,7 @@ static void test_http_no_content_length(void) {
   for (i = 0; i < 1000 && strchr(buf2, 'c') == NULL; i++) mg_mgr_poll(&mgr, 10);
   MG_INFO(("[%s] [%s]", buf1, buf2));
   ASSERT(strcmp(buf1, "mc") == 0);
-  ASSERT(strcmp(buf2, "cm") == 0);  // See #1475
+  ASSERT(strcmp(buf2, "mc") == 0);
   mg_mgr_free(&mgr);
   ASSERT(mgr.conns == NULL);
 }