changeset 8600:06af25901b1a quic

QUIC: restored proper usage of ngx_quic_drop_ack_ranges(). ACK Ranges are again managed based on the remembered Largest Acknowledged sent in the packet being acknowledged, which partially reverts c01964fd7b8b.
author Sergey Kandaurov <pluknet@nginx.com>
date Thu, 22 Oct 2020 11:05:50 +0100
parents c01964fd7b8b
children dd8e50e11bfc
files src/event/ngx_event_quic.c
diffstat 1 files changed, 4 insertions(+), 2 deletions(-) [+]
line wrap: on
line diff
--- a/src/event/ngx_event_quic.c
+++ b/src/event/ngx_event_quic.c
@@ -2799,8 +2799,6 @@ ngx_quic_handle_ack_frame(ngx_connection
         }
     }
 
-    ngx_quic_drop_ack_ranges(c, ctx, ack->largest);
-
     pos = ack->ranges_start;
     end = ack->ranges_end;
 
@@ -2871,6 +2869,10 @@ ngx_quic_handle_ack_frame_range(ngx_conn
             ngx_quic_congestion_ack(c, f);
 
             switch (f->type) {
+            case NGX_QUIC_FT_ACK:
+            case NGX_QUIC_FT_ACK_ECN:
+                ngx_quic_drop_ack_ranges(c, ctx, f->u.ack.largest);
+                break;
 
             case NGX_QUIC_FT_STREAM0:
             case NGX_QUIC_FT_STREAM1: