diff src/event/ngx_event_quic.c @ 8275:19660929e8ff quic

Logging cleanup. + Client-related errors (i.e. parsing) are done at INFO level + c->log->action is updated through the process of receiving, parsing. handling packet/payload and generating frames/output.
author Vladimir Homutov <vl@nginx.com>
date Tue, 24 Mar 2020 17:03:39 +0300
parents ee53bfd8f9ed
children 146687b1ab43
line wrap: on
line diff
--- a/src/event/ngx_event_quic.c
+++ b/src/event/ngx_event_quic.c
@@ -401,6 +401,8 @@ ngx_quic_new_connection(ngx_connection_t
         return NGX_ERROR;
     }
 
+    c->log->action = "creating new quic connection";
+
     qc = ngx_pcalloc(c->pool, sizeof(ngx_quic_connection_t));
     if (qc == NULL) {
         return NGX_ERROR;
@@ -792,7 +794,6 @@ ngx_quic_payload_handler(ngx_connection_
     ngx_quic_frame_t        frame, *ack_frame;
     ngx_quic_connection_t  *qc;
 
-    c->log->action = "processing quic payload";
 
     qc = c->quic;
 
@@ -804,6 +805,8 @@ ngx_quic_payload_handler(ngx_connection_
 
     while (p < end) {
 
+        c->log->action = "parsing frames";
+
         len = ngx_quic_parse_frame(pkt, p, end, &frame);
 
         if (len == NGX_DECLINED) {
@@ -817,6 +820,8 @@ ngx_quic_payload_handler(ngx_connection_
             return NGX_ERROR;
         }
 
+        c->log->action = "handling frames";
+
         p += len;
 
         switch (frame.type) {
@@ -932,6 +937,8 @@ ngx_quic_payload_handler(ngx_connection_
         return NGX_OK;
     }
 
+    c->log->action = "generating acknowledgment";
+
     // packet processed, ACK it now if required
     // TODO: if (ack_required) ...  - currently just ack each packet
 
@@ -1196,6 +1203,8 @@ ngx_quic_output(ngx_connection_t *c)
         return NGX_OK;
     }
 
+    c->log->action = "sending frames";
+
     lvl = qc->frames->level;
     start = qc->frames;
     f = start;