allow unknown frames

Author: marty1885

ChangeLog.md

@@ -1,10 +1,17 @@
-# Changelog
+#Changelog
 
 All notable changes to this project will be documented in this file.
 
 ## [Unreleased]
 
-## [1.10.0-beta.3] - TBD
+## [1.10.0-beta.4] - TBD
+
+### Fixes
+
+* Correctly track received bytes in HTTP/2 client statistics
+* Ignore unknown HTTP/2 frame types instead of treating them as fatal errors (forward compatibility)
+
+## [1.10.0-beta.3] - 2025-12-21
 
 ### Changes
 
@@ -425,27 +432,26 @@ All notable changes to this project will be documented in this file.
 
 - Added isTopicEmpty function;
 
-### Changed
+## #Changed
 
-- Update the ubuntu Dockerfile;
+    - Update the ubuntu Dockerfile;
 
 - Add optional Criteria && || operator support;
 
-- Bump actions/checkout from 3 to 4;
+- Bump actions / checkout from 3 to 4;
 
-- Make & and * directly adjacent to variable names;
+- Make & and *directly adjacent to variable names;
 
-- Use wss://echo.websocket.events/.ws in WebSocket client example;
+- Use wss:  // echo.websocket.events/.ws in WebSocket client example;
 
-- Change logs in the AccessLogger plugin to TRACE level;
+    -Change logs in the AccessLogger plugin to TRACE level;
 
-### Fixed
+## #Fixed
 
-- Fix an error in the secureRandomString function;
+    - Fix an error in the secureRandomString function;
 
 - FIX int mapping to int64_t instead of uint64_t;
 
-
 ## [1.9.0-rc.1] - 2023-09-23
 
 ### API changes list
@@ -1488,7 +1494,8 @@ All notable changes to this project will be documented in this file.
 
 - Destroy DNS resolver of HttpClient in the correct thread.
 
-- Add the header <cctype> to resolve build errors in VS2017.
+- Add the header <
+cctype & gt; to resolve build errors in VS2017.
 
 ## [1.0.0-beta18] - 2020-06-14
 
@@ -1830,7 +1837,7 @@ All notable changes to this project will be documented in this file.
 
 - Add two methods to control if the Server header or the Date header is sent to clients with HTTP responses.
     * void HttpAppFramework::enableServerHeader(bool);
-    * void HttpAppFramework::enableDateHeader(bool);
+*void HttpAppFramework::enableDateHeader(bool);
 
 ### Changed
 

lib/src/Http2Transport.cc

@@ -625,12 +625,13 @@ parseH2Frame(trantor::MsgBuffer *msg)
     // MSB is reserved for future use
     const uint32_t streamId = header.readU32BE() & ((1U << 31) - 1);
 
-    if (type >= (uint8_t)H2FrameType::NumEntries)
-    {
-        // TODO: Handle fatal protocol error
-        LOG_TRACE << "Unsupported H2 frame type: " << (int)type;
-        return {std::nullopt, streamId, 0, true, 0};
-    }
+    // This check is not need as HTTP/2 allows unknown frame types to be
+    // ignored for forward compatibility.
+    // if (type >= (uint8_t)H2FrameType::NumEntries)
+    // {
+    //     LOG_TRACE << "Unsupported H2 frame type: " << (int)type;
+    //     return {std::nullopt, streamId, 0, true, 0};
+    // }
 
     LOG_TRACE << "H2 frame: length=" << length << " type=" << (int)type
               << " flags=" << (int)flags << " streamId=" << streamId;
@@ -657,7 +658,7 @@ parseH2Frame(trantor::MsgBuffer *msg)
         frame = RstStreamFrame::parse(payload, flags);
     else
     {
-        LOG_WARN << "Unsupported H2 frame type: " << (int)type;
+        LOG_TRACE << "Unsupported H2 frame type: " << (int)type;
         msg->retrieve(length + 9);
         return {std::nullopt, streamId, 0, false, 0};
     }