This is an automated email from the ASF dual-hosted git repository.

markt-asf pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/tomcat.git


The following commit(s) were added to refs/heads/main by this push:
     new 27549bbc6f Review pr #1150
27549bbc6f is described below

commit 27549bbc6fed7cd1f3b1374261f29477d3f616b6
Author: Mark Thomas <[email protected]>
AuthorDate: Fri Nov 13 14:59:17 2026 +0000

    Review pr #1150
    
    - Update Javadoc
    - add change log entry
    - remove unnecessary code
---
 java/org/apache/tomcat/util/http/parser/EntityTag.java | 6 ++----
 webapps/docs/changelog.xml                             | 4 ++++
 2 files changed, 6 insertions(+), 4 deletions(-)

diff --git a/java/org/apache/tomcat/util/http/parser/EntityTag.java 
b/java/org/apache/tomcat/util/http/parser/EntityTag.java
index 7c0b4fb2bc..9b095348b3 100644
--- a/java/org/apache/tomcat/util/http/parser/EntityTag.java
+++ b/java/org/apache/tomcat/util/http/parser/EntityTag.java
@@ -32,8 +32,8 @@ public class EntityTag {
     }
 
     /**
-     * Parse the given input as (per RFC 7232) 1#entity-tag. Compare an ETag 
header with a resource ETag as described in
-     * RFC 7232 section 2.3.2.
+     * Parse the given input as (per RFC 9110) #entity-tag. Compare an ETag 
header with a resource ETag as described in
+     * RFC 9110 section 8.8.3.2.
      *
      * @param input        The input to parse
      * @param compareWeak  Use weak comparison e.g. match "etag" with W/"etag"
@@ -92,8 +92,6 @@ public class EntityTag {
                 }
             }
 
-            HttpParser.skipWhitespace(input);
-
             switch (HttpParser.skipConstant(input, ",")) {
                 case EOF:
                     return result;
diff --git a/webapps/docs/changelog.xml b/webapps/docs/changelog.xml
index f990bb038b..f7aa3b3eb5 100644
--- a/webapps/docs/changelog.xml
+++ b/webapps/docs/changelog.xml
@@ -303,6 +303,10 @@
         section 15.2). Pull request <pr>1149</pr> provided by Arturo Bernal.
         (markt)
       </fix>
+      <fix>
+        Skip empty elements when parsing entity tag lists. Pull request
+        <pr>1150</pr> provided by Arturo Bernal. (markt)
+      </fix>
     </changelog>
   </subsection>
   <subsection name="Jasper">


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to