This is an automated email from the ASF dual-hosted git repository.
remm pushed a commit to branch 9.0.x
in repository https://gitbox.apache.org/repos/asf/tomcat.git
The following commit(s) were added to refs/heads/9.0.x by this push:
new 82b088d749 Fix evaluation order in the old code
82b088d749 is described below
commit 82b088d749c145b4eeb26b27f96f0584e87159f2
Author: remm <[email protected]>
AuthorDate: Thu Dec 12 10:42:06 2024 +0100
Fix evaluation order in the old code
Format to make the order more visible.
Test submitted by Chenjp.
---
java/org/apache/catalina/servlets/DefaultServlet.java | 6 ++++--
.../servlets/TestDefaultServletRfc9110Section13Parameterized.java | 2 ++
2 files changed, 6 insertions(+), 2 deletions(-)
diff --git a/java/org/apache/catalina/servlets/DefaultServlet.java
b/java/org/apache/catalina/servlets/DefaultServlet.java
index cc7fc23f69..a08971e883 100644
--- a/java/org/apache/catalina/servlets/DefaultServlet.java
+++ b/java/org/apache/catalina/servlets/DefaultServlet.java
@@ -749,8 +749,10 @@ public class DefaultServlet extends HttpServlet {
*/
protected boolean checkIfHeaders(HttpServletRequest request,
HttpServletResponse response, WebResource resource)
throws IOException {
- return checkIfMatch(request, response, resource) &&
checkIfModifiedSince(request, response, resource) &&
- checkIfNoneMatch(request, response, resource) &&
checkIfUnmodifiedSince(request, response, resource);
+ return checkIfMatch(request, response, resource)
+ && checkIfUnmodifiedSince(request, response, resource)
+ && checkIfNoneMatch(request, response, resource)
+ && checkIfModifiedSince(request, response, resource);
}
diff --git
a/test/org/apache/catalina/servlets/TestDefaultServletRfc9110Section13Parameterized.java
b/test/org/apache/catalina/servlets/TestDefaultServletRfc9110Section13Parameterized.java
index d349a0b075..0c0f6771a9 100644
---
a/test/org/apache/catalina/servlets/TestDefaultServletRfc9110Section13Parameterized.java
+++
b/test/org/apache/catalina/servlets/TestDefaultServletRfc9110Section13Parameterized.java
@@ -103,6 +103,8 @@ public class
TestDefaultServletRfc9110Section13Parameterized extends TomcatBaseT
Boolean.FALSE, null, new int[] { 200 } });
parameterSets.add(new Object[] { Boolean.TRUE, Task.HEAD_INDEX_HTML,
null, IfPolicy.DATE_MULTI_IN, null, null,
null, Boolean.FALSE, null, new int[] { 200 } });
+ parameterSets.add(new Object[] { Boolean.TRUE, Task.HEAD_INDEX_HTML,
null, IfPolicy.DATE_LT, null, IfPolicy.DATE_GT,
+ null, Boolean.FALSE, null, new int[] { 412 } });
parameterSets.add(new Object[] { Boolean.FALSE, Task.HEAD_INDEX_HTML,
null, IfPolicy.DATE_EQ, null, null, null,
Boolean.FALSE, null, new int[] { 200 } });
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
