changeset 8605:eed49b83e18f quic

QUIC: revised value separators in debug and error messages. All values are prefixed with name and separated from it using colon. Multiple values are listed without commas in between. Rationale: this greatly simplifies log parsing for analysis.
author Vladimir Homutov <vl@nginx.com>
date Tue, 27 Oct 2020 14:12:31 +0300
parents b3d9e57d0f62
children c53761445a66
files src/event/ngx_event_quic.c src/event/ngx_event_quic_protection.c src/event/ngx_event_quic_transport.c
diffstat 3 files changed, 72 insertions(+), 69 deletions(-) [+]
line wrap: on
line diff
--- a/src/event/ngx_event_quic.c
+++ b/src/event/ngx_event_quic.c
@@ -699,7 +699,7 @@ ngx_quic_add_handshake_data(ngx_ssl_conn
 
         ngx_log_debug1(NGX_LOG_DEBUG_EVENT, c->log, 0,
                        "quic SSL_get_peer_quic_transport_params():"
-                       " params_len %ui", client_params_len);
+                       " params_len:%ui", client_params_len);
 
         if (client_params_len == 0) {
             /* quic-tls 8.2 */
@@ -828,7 +828,7 @@ ngx_quic_send_alert(ngx_ssl_conn_t *ssl_
     c = ngx_ssl_get_connection((ngx_ssl_conn_t *) ssl_conn);
 
     ngx_log_debug2(NGX_LOG_DEBUG_EVENT, c->log, 0,
-                   "quic ngx_quic_send_alert(), lvl=%d, alert=%d",
+                   "quic ngx_quic_send_alert() lvl:%d  alert:%d",
                    (int) level, (int) alert);
 
     qc = c->quic;
@@ -1582,7 +1582,7 @@ ngx_quic_close_connection(ngx_connection
     ngx_pool_t  *pool;
 
     ngx_log_debug1(NGX_LOG_DEBUG_EVENT, c->log, 0,
-                   "quic ngx_quic_close_connection, rc: %i", rc);
+                   "quic ngx_quic_close_connection rc:%i", rc);
 
     if (!c->quic) {
         ngx_log_debug0(NGX_LOG_DEBUG_EVENT, c->log, 0,
@@ -1658,7 +1658,7 @@ ngx_quic_close_quic(ngx_connection_t *c,
 
             if (rc == NGX_OK) {
                  ngx_log_debug1(NGX_LOG_DEBUG_EVENT, c->log, 0,
-                                "quic immediate close, drain = %d",
+                                "quic immediate close drain:%d",
                                 qc->draining);
 
                 qc->close.log = c->log;
@@ -1678,7 +1678,7 @@ ngx_quic_close_quic(ngx_connection_t *c,
                 }
 
                 ngx_log_debug3(NGX_LOG_DEBUG_EVENT, c->log, 0,
-                               "quic immediate close due to %serror: %ui %s",
+                               "quic immediate close due to %s error: %ui %s",
                                qc->error_app ? "app " : "", qc->error,
                                qc->error_reason ? qc->error_reason : "");
             }
@@ -2019,7 +2019,7 @@ ngx_quic_process_packet(ngx_connection_t
                 /* 7.2.  Negotiating Connection IDs */
                 ngx_log_error(NGX_LOG_INFO, c->log, 0,
                               "quic too short dcid in initial"
-                              " packet: length %i", pkt->dcid.len);
+                              " packet: len:%i", pkt->dcid.len);
                 return NGX_ERROR;
             }
 
@@ -2465,7 +2465,7 @@ ngx_quic_payload_handler(ngx_connection_
 
     if (p != end) {
         ngx_log_error(NGX_LOG_INFO, c->log, 0,
-                      "quic trailing garbage in payload: %ui bytes", end - p);
+                      "quic trailing garbage in payload:%ui bytes", end - p);
 
         qc->error = NGX_QUIC_ERR_FRAME_ENCODING_ERROR;
         return NGX_ERROR;
@@ -2498,8 +2498,8 @@ ngx_quic_ack_packet(ngx_connection_t *c,
     ctx = ngx_quic_get_send_ctx(c->quic, pkt->level);
 
     ngx_log_debug4(NGX_LOG_DEBUG_EVENT, c->log, 0,
-                   "quic ngx_quic_ack_packet pn %uL largest %uL"
-                   " first %uL nranges %ui", pkt->pn, ctx->largest_range,
+                   "quic ngx_quic_ack_packet pn:%uL largest %uL fr:%uL"
+                   " nranges:%ui", pkt->pn, ctx->largest_range,
                    ctx->first_range, ctx->nranges);
 
     prev_pending = ctx->pending_ack;
@@ -2750,8 +2750,8 @@ ngx_quic_drop_ack_ranges(ngx_connection_
     ngx_quic_ack_range_t  *r;
 
     ngx_log_debug4(NGX_LOG_DEBUG_EVENT, c->log, 0,
-                   "quic ngx_quic_drop_ack_ranges pn %uL largest %uL"
-                   " first %uL nranges %ui", pn, ctx->largest_range,
+                   "quic ngx_quic_drop_ack_ranges pn:%uL largest:%uL"
+                   " fr:%uL nranges:%ui", pn, ctx->largest_range,
                    ctx->first_range, ctx->nranges);
 
     base = ctx->largest_range;
@@ -2936,7 +2936,7 @@ ngx_quic_handle_ack_frame(ngx_connection
     ctx = ngx_quic_get_send_ctx(qc, pkt->level);
 
     ngx_log_debug1(NGX_LOG_DEBUG_EVENT, c->log, 0,
-                   "quic ngx_quic_handle_ack_frame level %d", pkt->level);
+                   "quic ngx_quic_handle_ack_frame level:%d", pkt->level);
 
     /*
      *  If any computed packet number is negative, an endpoint MUST
@@ -2964,7 +2964,7 @@ ngx_quic_handle_ack_frame(ngx_connection
     if (ctx->largest_ack < max || ctx->largest_ack == NGX_QUIC_UNSET_PN) {
         ctx->largest_ack = max;
         ngx_log_debug1(NGX_LOG_DEBUG_EVENT, c->log, 0,
-                       "quic updated largest received ack: %uL", max);
+                       "quic updated largest received ack:%uL", max);
 
         /*
          *  An endpoint generates an RTT sample on receiving an
@@ -2993,7 +2993,7 @@ ngx_quic_handle_ack_frame(ngx_connection
         if (gap + 2 > min) {
             qc->error = NGX_QUIC_ERR_FRAME_ENCODING_ERROR;
             ngx_log_error(NGX_LOG_INFO, c->log, 0,
-                         "quic invalid range %ui in ack frame", i);
+                         "quic invalid range:%ui in ack frame", i);
             return NGX_ERROR;
         }
 
@@ -3002,7 +3002,7 @@ ngx_quic_handle_ack_frame(ngx_connection
         if (range > max) {
             qc->error = NGX_QUIC_ERR_FRAME_ENCODING_ERROR;
             ngx_log_error(NGX_LOG_INFO, c->log, 0,
-                         "quic invalid range %ui in ack frame", i);
+                         "quic invalid range:%ui in ack frame", i);
             return NGX_ERROR;
         }
 
@@ -3030,7 +3030,7 @@ ngx_quic_handle_ack_frame_range(ngx_conn
     ngx_quic_connection_t  *qc;
 
     ngx_log_debug2(NGX_LOG_DEBUG_EVENT, c->log, 0,
-                   "quic handle ack range: min:%uL max:%uL", min, max);
+                   "quic handle ack range min:%uL max:%uL", min, max);
 
     qc = c->quic;
 
@@ -3145,7 +3145,7 @@ ngx_quic_rtt_sample(ngx_connection_t *c,
     }
 
     ngx_log_debug4(NGX_LOG_DEBUG_EVENT, c->log, 0,
-                   "quic rtt sample: latest %M, min %M, avg %M, var %M",
+                   "quic rtt sample latest:%M min:%M avg:%M var:%M",
                    latest_rtt, qc->min_rtt, qc->avg_rtt, qc->rttvar);
 }
 
@@ -3205,7 +3205,7 @@ ngx_quic_handle_stream_ack(ngx_connectio
     sn->acked += f->u.stream.length;
 
     ngx_log_debug3(NGX_LOG_DEBUG_EVENT, sn->c->log, 0,
-                   "quic stream ack %uL acked:%uL, unacked:%uL",
+                   "quic stream ack len:%uL acked:%uL unacked:%uL",
                    f->u.stream.length, sn->acked, sent - sn->acked);
 }
 
@@ -3223,7 +3223,7 @@ ngx_quic_handle_ordered_frame(ngx_connec
 
     if (f->offset > fs->received) {
         ngx_log_debug2(NGX_LOG_DEBUG_EVENT, c->log, 0,
-                       "quic out-of-order frame: expecting %uL got %uL",
+                       "quic out-of-order frame: expecting:%uL got:%uL",
                        fs->received, f->offset);
 
         return ngx_quic_buffer_frame(c, fs, frame);
@@ -3284,7 +3284,7 @@ ngx_quic_handle_ordered_frame(ngx_connec
                 fs->total -= f->length;
 
                 ngx_log_debug1(NGX_LOG_DEBUG_EVENT, c->log, 0,
-                              "quic skipped buffered frame, total %ui",
+                              "quic skipped buffered frame, total:%ui",
                               fs->total);
                 ngx_quic_free_frame(c, frame);
                 continue;
@@ -3311,7 +3311,7 @@ ngx_quic_handle_ordered_frame(ngx_connec
         ngx_queue_remove(q);
 
         ngx_log_debug1(NGX_LOG_DEBUG_EVENT, c->log, 0,
-                      "quic consumed buffered frame, total %ui", fs->total);
+                      "quic consumed buffered frame, total:%ui", fs->total);
 
         ngx_quic_free_frame(c, frame);
 
@@ -3387,7 +3387,7 @@ ngx_quic_buffer_frame(ngx_connection_t *
 
     ngx_log_debug1(NGX_LOG_DEBUG_EVENT, c->log, 0,
                   "quic ordered frame with unexpected offset:"
-                  " buffered, total %ui", fs->total);
+                  " buffered total:%ui", fs->total);
 
     if (ngx_queue_empty(&fs->frames)) {
         ngx_queue_insert_after(&fs->frames, &dst->queue);
@@ -3468,7 +3468,7 @@ ngx_quic_crypto_input(ngx_connection_t *
     ssl_conn = c->ssl->connection;
 
     ngx_log_debug2(NGX_LOG_DEBUG_EVENT, c->log, 0,
-                   "quic SSL_quic_read_level: %d, SSL_quic_write_level: %d",
+                   "quic SSL_quic_read_level:%d SSL_quic_write_level:%d",
                    (int) SSL_quic_read_level(ssl_conn),
                    (int) SSL_quic_write_level(ssl_conn));
 
@@ -3498,7 +3498,7 @@ ngx_quic_crypto_input(ngx_connection_t *
     } else if (n == 1 && !SSL_in_init(ssl_conn)) {
 
         ngx_log_debug1(NGX_LOG_DEBUG_EVENT, c->log, 0,
-                       "quic ssl cipher: %s", SSL_get_cipher(ssl_conn));
+                       "quic ssl cipher:%s", SSL_get_cipher(ssl_conn));
 
         ngx_log_debug0(NGX_LOG_DEBUG_EVENT, c->log, 0,
                        "quic handshake completed successfully");
@@ -3540,7 +3540,7 @@ ngx_quic_crypto_input(ngx_connection_t *
     }
 
     ngx_log_debug2(NGX_LOG_DEBUG_EVENT, c->log, 0,
-                   "quic SSL_quic_read_level: %d, SSL_quic_write_level: %d",
+                   "quic SSL_quic_read_level:%d SSL_quic_write_level:%d",
                    (int) SSL_quic_read_level(ssl_conn),
                    (int) SSL_quic_write_level(ssl_conn));
 
@@ -4330,7 +4330,7 @@ ngx_quic_output_frames(ngx_connection_t 
                 if (((c->sent + hlen + len + f->len) / 3) > qc->received) {
                     ngx_log_debug2(NGX_LOG_DEBUG_EVENT, c->log, 0,
                                    "quic hit amplification limit"
-                                   " received %uz sent %O",
+                                   " received:%uz sent:%O",
                                    qc->received, c->sent);
                     break;
                 }
@@ -4488,8 +4488,8 @@ ngx_quic_send_frames(ngx_connection_t *c
     res.data = dst;
 
     ngx_log_debug6(NGX_LOG_DEBUG_EVENT, c->log, 0,
-                   "quic packet ready: %ui bytes at level %d"
-                   " need_ack: %d number: %L encoded %d:0x%xD",
+                   "quic packet ready bytes:%ui level:%d"
+                   " need_ack:%d number:%L encoded nl:%d trunc:0x%xD",
                    out.len, start->level, pkt.need_ack, pkt.number,
                    pkt.num_len, pkt.trunc);
 
@@ -4771,7 +4771,7 @@ ngx_quic_open_stream(ngx_connection_t *c
             >= qc->streams.server_max_streams_bidi)
         {
             ngx_log_debug1(NGX_LOG_DEBUG_EVENT, c->log, 0,
-                           "quic too many server bidi streams: %uL",
+                           "quic too many server bidi streams:%uL",
                            qc->streams.server_streams_bidi);
             return NULL;
         }
@@ -4780,7 +4780,8 @@ ngx_quic_open_stream(ngx_connection_t *c
              | NGX_QUIC_STREAM_SERVER_INITIATED;
 
         ngx_log_debug3(NGX_LOG_DEBUG_EVENT, c->log, 0,
-                       "quic creating server bidi stream %uL/%uL id:0x%xL",
+                       "quic creating server bidi stream"
+                       " streams:%uL max:%uL id:0x%xL",
                        qc->streams.server_streams_bidi,
                        qc->streams.server_max_streams_bidi, id);
 
@@ -4792,7 +4793,7 @@ ngx_quic_open_stream(ngx_connection_t *c
             >= qc->streams.server_max_streams_uni)
         {
             ngx_log_debug1(NGX_LOG_DEBUG_EVENT, c->log, 0,
-                           "quic too many server uni streams: %uL",
+                           "quic too many server uni streams:%uL",
                            qc->streams.server_streams_uni);
             return NULL;
         }
@@ -4802,7 +4803,8 @@ ngx_quic_open_stream(ngx_connection_t *c
              | NGX_QUIC_STREAM_UNIDIRECTIONAL;
 
         ngx_log_debug3(NGX_LOG_DEBUG_EVENT, c->log, 0,
-                       "quic creating server uni stream %uL/%uL id:0x%xL",
+                       "quic creating server uni stream"
+                       " streams:%uL max:%uL id:0x%xL",
                        qc->streams.server_streams_uni,
                        qc->streams.server_max_streams_uni, id);
 
@@ -4879,7 +4881,7 @@ ngx_quic_create_client_stream(ngx_connec
     ngx_quic_connection_t  *qc;
 
     ngx_log_debug1(NGX_LOG_DEBUG_EVENT, c->log, 0,
-                   "quic stream id 0x%xL is new", id);
+                   "quic stream id:0x%xL is new", id);
 
     qc = c->quic;
 
@@ -4970,7 +4972,7 @@ ngx_quic_create_stream(ngx_connection_t 
     ngx_quic_connection_t  *qc;
 
     ngx_log_debug1(NGX_LOG_DEBUG_EVENT, c->log, 0,
-                   "quic stream id 0x%xL create", id);
+                   "quic stream id:0x%xL create", id);
 
     qc = c->quic;
 
@@ -5089,7 +5091,7 @@ ngx_quic_stream_recv(ngx_connection_t *c
     }
 
     ngx_log_debug3(NGX_LOG_DEBUG_EVENT, c->log, 0,
-                   "quic stream id 0x%xL recv: eof:%d, avail:%z",
+                   "quic stream recv id:0x%xL eof:%d avail:%z",
                    qs->id, rev->pending_eof, b->last - b->pos);
 
     if (b->pos == b->last) {
@@ -5101,7 +5103,7 @@ ngx_quic_stream_recv(ngx_connection_t *c
         }
 
         ngx_log_debug1(NGX_LOG_DEBUG_EVENT, c->log, 0,
-                       "quic stream id 0x%xL recv() not ready", qs->id);
+                       "quic stream id:0x%xL recv() not ready", qs->id);
         return NGX_AGAIN;
     }
 
@@ -5119,7 +5121,8 @@ ngx_quic_stream_recv(ngx_connection_t *c
     }
 
     ngx_log_debug3(NGX_LOG_DEBUG_EVENT, c->log, 0,
-                   "quic stream id 0x%xL recv: %z of %uz", qs->id, len, size);
+                   "quic stream id:0x%xL recv len:%z of size:%uz",
+                   qs->id, len, size);
 
     if (!rev->pending_eof) {
         frame = ngx_quic_alloc_frame(pc, 0);
@@ -5153,7 +5156,7 @@ ngx_quic_stream_recv(ngx_connection_t *c
         ngx_quic_queue_frame(pc->quic, frame);
 
         ngx_log_debug2(NGX_LOG_DEBUG_EVENT, c->log, 0,
-                       "quic stream id 0x%xL recv: increased max data: %uL",
+                       "quic stream id:0x%xL recv: increased max_data:%uL",
                        qs->id, qc->streams.recv_max_data);
     }
 
@@ -5299,7 +5302,7 @@ ngx_quic_stream_send_chain(ngx_connectio
     }
 
     ngx_log_debug2(NGX_LOG_DEBUG_EVENT, c->log, 0,
-                   "quic send_chain sent:%uz, frames:%ui", sent, nframes);
+                   "quic send_chain sent:%uz nframes:%ui", sent, nframes);
 
     return in;
 }
@@ -5368,7 +5371,7 @@ ngx_quic_max_stream_flow(ngx_connection_
     }
 
     ngx_log_debug1(NGX_LOG_DEBUG_EVENT, c->log, 0,
-                   "quic send flow: %uz", size);
+                   "quic send flow:%uz", size);
 
     return size;
 }
@@ -5389,7 +5392,7 @@ ngx_quic_stream_cleanup_handler(void *da
     qc = pc->quic;
 
     ngx_log_debug1(NGX_LOG_DEBUG_EVENT, c->log, 0,
-                   "quic stream id 0x%xL cleanup", qs->id);
+                   "quic stream id:0x%xL cleanup", qs->id);
 
     ngx_rbtree_delete(&qc->streams.tree, &qs->node);
     ngx_quic_free_frames(pc, &qs->fs.frames);
@@ -5445,7 +5448,7 @@ ngx_quic_stream_cleanup_handler(void *da
     }
 
     ngx_log_debug1(NGX_LOG_DEBUG_EVENT, c->log, 0,
-                   "quic stream id 0x%xL send fin", qs->id);
+                   "quic stream id:0x%xL send fin", qs->id);
 
     frame = ngx_quic_alloc_frame(pc, 0);
     if (frame == NULL) {
@@ -5547,7 +5550,7 @@ ngx_quic_congestion_ack(ngx_connection_t
 
     if ((ngx_msec_int_t) timer <= 0) {
         ngx_log_debug3(NGX_LOG_DEBUG_EVENT, c->log, 0,
-                       "quic congestion ack recovery win:%uz, ss:%uz, if:%uz",
+                       "quic congestion ack recovery win:%uz ss:%uz if:%uz",
                        cg->window, cg->ssthresh, cg->in_flight);
 
         return;
@@ -5557,14 +5560,14 @@ ngx_quic_congestion_ack(ngx_connection_t
         cg->window += f->plen;
 
         ngx_log_debug3(NGX_LOG_DEBUG_EVENT, c->log, 0,
-                       "quic congestion slow start win:%uz, ss:%uz, if:%uz",
+                       "quic congestion slow start win:%uz ss:%uz if:%uz",
                        cg->window, cg->ssthresh, cg->in_flight);
 
     } else {
         cg->window += qc->tp.max_udp_payload_size * f->plen / cg->window;
 
         ngx_log_debug3(NGX_LOG_DEBUG_EVENT, c->log, 0,
-                       "quic congestion avoidance win:%uz, ss:%uz, if:%uz",
+                       "quic congestion avoidance win:%uz ss:%uz if:%uz",
                        cg->window, cg->ssthresh, cg->in_flight);
     }
 
@@ -5598,7 +5601,7 @@ ngx_quic_congestion_lost(ngx_connection_
 
     if ((ngx_msec_int_t) timer <= 0) {
         ngx_log_debug3(NGX_LOG_DEBUG_EVENT, c->log, 0,
-                       "quic congestion lost recovery win:%uz, ss:%uz, if:%uz",
+                       "quic congestion lost recovery win:%uz ss:%uz if:%uz",
                        cg->window, cg->ssthresh, cg->in_flight);
 
         return;
@@ -5614,7 +5617,7 @@ ngx_quic_congestion_lost(ngx_connection_
     cg->ssthresh = cg->window;
 
     ngx_log_debug3(NGX_LOG_DEBUG_EVENT, c->log, 0,
-                   "quic congestion lost win:%uz, ss:%uz, if:%uz",
+                   "quic congestion lost win:%uz ss:%uz if:%uz",
                    cg->window, cg->ssthresh, cg->in_flight);
 }
 
--- a/src/event/ngx_event_quic_protection.c
+++ b/src/event/ngx_event_quic_protection.c
@@ -1113,9 +1113,9 @@ ngx_quic_decrypt(ngx_quic_header_t *pkt,
     pkt->flags = clearflags;
 
     ngx_log_debug1(NGX_LOG_DEBUG_EVENT, pkt->log, 0,
-                   "quic clear flags: %xd", clearflags);
+                   "quic clearflags:%xd", clearflags);
     ngx_log_debug2(NGX_LOG_DEBUG_EVENT, pkt->log, 0,
-                   "quic packet number: %uL, len: %xi", pn, pnl);
+                   "quic packet number:%uL len:%xi", pn, pnl);
 
     /* packet protection */
 
--- a/src/event/ngx_event_quic_transport.c
+++ b/src/event/ngx_event_quic_transport.c
@@ -597,7 +597,7 @@ ngx_quic_parse_initial_header(ngx_quic_h
     }
 
     ngx_log_debug1(NGX_LOG_DEBUG_EVENT, pkt->log, 0,
-                   "quic initial packet length: %uL", varint);
+                   "quic initial packet len:%uL", varint);
 
     if (varint > (uint64_t) ((pkt->data + pkt->len) - p)) {
         ngx_log_error(NGX_LOG_INFO, pkt->log, 0,
@@ -636,7 +636,7 @@ ngx_quic_parse_handshake_header(ngx_quic
     }
 
     ngx_log_debug1(NGX_LOG_DEBUG_EVENT, pkt->log, 0,
-                   "quic handshake packet length: %uL", plen);
+                   "quic handshake packet len:%uL", plen);
 
     if (plen > (uint64_t)((pkt->data + pkt->len) - p)) {
         ngx_log_error(NGX_LOG_INFO, pkt->log, 0,
@@ -747,7 +747,7 @@ ngx_quic_parse_frame(ngx_quic_header_t *
             }
 
             ngx_log_debug3(NGX_LOG_DEBUG_EVENT, pkt->log, 0,
-                           "quic ACK ECN counters: %uL %uL %uL",
+                           "quic ACK ECN counters ect0:%uL ect1:%uL ce:%uL",
                            f->u.ack.ect0, f->u.ack.ect1, f->u.ack.ce);
         }
 
@@ -1013,7 +1013,7 @@ error:
     pkt->error = NGX_QUIC_ERR_FRAME_ENCODING_ERROR;
 
     ngx_log_error(NGX_LOG_INFO, pkt->log, 0,
-                  "quic failed to parse frame type 0x%xi", f->type);
+                  "quic failed to parse frame type:0x%xi", f->type);
 
     return NGX_ERROR;
 }
@@ -1509,7 +1509,7 @@ ngx_quic_parse_transport_params(u_char *
         case NGX_QUIC_TP_SR_TOKEN:
             ngx_log_error(NGX_LOG_INFO, log, 0,
                           "quic client sent forbidden transport param"
-                          " id 0x%xL", id);
+                          " id:0x%xL", id);
             return NGX_ERROR;
         }
 
@@ -1517,7 +1517,7 @@ ngx_quic_parse_transport_params(u_char *
         if (p == NULL) {
             ngx_log_error(NGX_LOG_INFO, log, 0,
                          "quic failed to parse"
-                         " transport param id 0x%xL length", id);
+                         " transport param id:0x%xL length", id);
             return NGX_ERROR;
         }
 
@@ -1526,13 +1526,13 @@ ngx_quic_parse_transport_params(u_char *
         if (rc == NGX_ERROR) {
             ngx_log_error(NGX_LOG_INFO, log, 0,
                           "quic failed to parse"
-                          " transport param id 0x%xL data", id);
+                          " transport param id:0x%xL data", id);
             return NGX_ERROR;
         }
 
         if (rc == NGX_DECLINED) {
             ngx_log_error(NGX_LOG_INFO, log, 0,
-                          "quic unknown transport param id 0x%xL, skipped", id);
+                          "quic unknown transport param id:0x%xL, skipped", id);
         }
 
         p += len;
@@ -1541,7 +1541,7 @@ ngx_quic_parse_transport_params(u_char *
     if (p != end) {
         ngx_log_error(NGX_LOG_INFO, log, 0,
                       "quic trailing garbage in"
-                      " transport parameters: %ui bytes",
+                      " transport parameters: bytes:%ui",
                       end - p);
         return NGX_ERROR;
     }
@@ -1553,45 +1553,45 @@ ngx_quic_parse_transport_params(u_char *
                    "quic tp disable active migration: %ui",
                    tp->disable_active_migration);
 
-    ngx_log_debug1(NGX_LOG_DEBUG_EVENT, log, 0, "quic tp idle_timeout: %ui",
+    ngx_log_debug1(NGX_LOG_DEBUG_EVENT, log, 0, "quic tp idle_timeout:%ui",
                    tp->max_idle_timeout);
 
     ngx_log_debug1(NGX_LOG_DEBUG_EVENT, log, 0,
-                   "quic tp max_udp_payload_size: %ui",
+                   "quic tp max_udp_payload_size:%ui",
                    tp->max_udp_payload_size);
 
-    ngx_log_debug1(NGX_LOG_DEBUG_EVENT, log, 0, "quic tp max_data: %ui",
+    ngx_log_debug1(NGX_LOG_DEBUG_EVENT, log, 0, "quic tp max_data:%ui",
                    tp->initial_max_data);
 
     ngx_log_debug1(NGX_LOG_DEBUG_EVENT, log, 0,
-                   "quic tp max_stream_data_bidi_local: %ui",
+                   "quic tp max_stream_data_bidi_local:%ui",
                    tp->initial_max_stream_data_bidi_local);
 
     ngx_log_debug1(NGX_LOG_DEBUG_EVENT, log, 0,
-                   "quic tp max_stream_data_bidi_remote: %ui",
+                   "quic tp max_stream_data_bidi_remote:%ui",
                    tp->initial_max_stream_data_bidi_remote);
 
     ngx_log_debug1(NGX_LOG_DEBUG_EVENT, log, 0,
-                   "quic tp max_stream_data_uni: %ui",
+                   "quic tp max_stream_data_uni:%ui",
                    tp->initial_max_stream_data_uni);
 
     ngx_log_debug1(NGX_LOG_DEBUG_EVENT, log, 0,
-                   "quic tp initial_max_streams_bidi: %ui",
+                   "quic tp initial_max_streams_bidi:%ui",
                    tp->initial_max_streams_bidi);
 
     ngx_log_debug1(NGX_LOG_DEBUG_EVENT, log, 0,
-                   "quic tp initial_max_streams_uni: %ui",
+                   "quic tp initial_max_streams_uni:%ui",
                    tp->initial_max_streams_uni);
 
     ngx_log_debug1(NGX_LOG_DEBUG_EVENT, log, 0,
-                   "quic tp ack_delay_exponent: %ui",
+                   "quic tp ack_delay_exponent:%ui",
                    tp->ack_delay_exponent);
 
-    ngx_log_debug1(NGX_LOG_DEBUG_EVENT, log, 0, "quic tp max_ack_delay: %ui",
+    ngx_log_debug1(NGX_LOG_DEBUG_EVENT, log, 0, "quic tp max_ack_delay:%ui",
                    tp->max_ack_delay);
 
     ngx_log_debug1(NGX_LOG_DEBUG_EVENT, log, 0,
-                   "quic tp active_connection_id_limit: %ui",
+                   "quic tp active_connection_id_limit:%ui",
                    tp->active_connection_id_limit);
 
 #if (NGX_QUIC_DRAFT_VERSION >= 28)