This is an automated email from the ASF dual-hosted git repository.
dsoumis pushed a commit to branch 10.1.x
in repository https://gitbox.apache.org/repos/asf/tomcat.git
The following commit(s) were added to refs/heads/10.1.x by this push:
new 556122adef Refactor CORS validation order with explicit guard clauses
556122adef is described below
commit 556122adef739d45247972bd6e68d69f96a10037
Author: Dimitris Soumis <jimsou...@gmail.com>
AuthorDate: Fri Feb 14 12:24:28 2025 +0200
Refactor CORS validation order with explicit guard clauses
- Check for null originHeader first to prevent NPEs.
- Validate empty or invalid originHeader before performing same-origin
checks.
---
java/org/apache/catalina/filters/CorsFilter.java | 7 +++++--
1 file changed, 5 insertions(+), 2 deletions(-)
diff --git a/java/org/apache/catalina/filters/CorsFilter.java
b/java/org/apache/catalina/filters/CorsFilter.java
index f96d26fda0..7363aa9f62 100644
--- a/java/org/apache/catalina/filters/CorsFilter.java
+++ b/java/org/apache/catalina/filters/CorsFilter.java
@@ -542,12 +542,15 @@ public class CorsFilter extends GenericFilter {
throw new
IllegalArgumentException(sm.getString("corsFilter.nullRequest"));
}
String originHeader = request.getHeader(REQUEST_HEADER_ORIGIN);
- if (originHeader == null || RequestUtil.isSameOrigin(request,
originHeader)) {
+ if (originHeader == null) {
return CORSRequestType.NOT_CORS;
}
- if (originHeader.isEmpty() ||!RequestUtil.isValidOrigin(originHeader))
{
+ if (originHeader.isEmpty() ||
!RequestUtil.isValidOrigin(originHeader)) {
return CORSRequestType.INVALID_CORS;
}
+ if(RequestUtil.isSameOrigin(request, originHeader)) {
+ return CORSRequestType.NOT_CORS;
+ }
String method = request.getMethod();
if (method == null) {
return CORSRequestType.INVALID_CORS;
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
For additional commands, e-mail: dev-h...@tomcat.apache.org
