This is an automated email from the ASF dual-hosted git repository.
adoroszlai pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/ozone.git
The following commit(s) were added to refs/heads/master by this push:
new 1e472174f7 HDDS-13168. Fix error response format in
CheckUploadContentTypeFilter (#8614)
1e472174f7 is described below
commit 1e472174f7dcfe6b29d0c6c6864fe61f9f11c521
Author: Eric Chang <[email protected]>
AuthorDate: Fri Jun 13 12:37:49 2025 +0800
HDDS-13168. Fix error response format in CheckUploadContentTypeFilter
(#8614)
---
.../http/server/CheckUploadContentTypeFilter.java | 21 ++++++++++++++++++---
1 file changed, 18 insertions(+), 3 deletions(-)
diff --git
a/hadoop-ozone/httpfsgateway/src/main/java/org/apache/ozone/fs/http/server/CheckUploadContentTypeFilter.java
b/hadoop-ozone/httpfsgateway/src/main/java/org/apache/ozone/fs/http/server/CheckUploadContentTypeFilter.java
index a4a1682138..56e4ad38d4 100644
---
a/hadoop-ozone/httpfsgateway/src/main/java/org/apache/ozone/fs/http/server/CheckUploadContentTypeFilter.java
+++
b/hadoop-ozone/httpfsgateway/src/main/java/org/apache/ozone/fs/http/server/CheckUploadContentTypeFilter.java
@@ -18,7 +18,10 @@
package org.apache.ozone.fs.http.server;
import java.io.IOException;
+import java.io.PrintWriter;
+import java.util.HashMap;
import java.util.HashSet;
+import java.util.Map;
import java.util.Set;
import javax.servlet.Filter;
import javax.servlet.FilterChain;
@@ -93,10 +96,22 @@ public void doFilter(ServletRequest request,
ServletResponse response,
if (contentTypeOK) {
chain.doFilter(httpReq, httpRes);
} else {
- httpRes.sendError(HttpServletResponse.SC_BAD_REQUEST,
- "Data upload requests must have content-type set to '"
+
- HttpFSConstants.UPLOAD_CONTENT_TYPE + "'");
+ httpRes.setStatus(HttpServletResponse.SC_BAD_REQUEST);
+ httpRes.setContentType("application/json");
+ httpRes.setCharacterEncoding("UTF-8");
+ String errorMessage = "Data upload requests must have content-type set
to '" +
+ HttpFSConstants.UPLOAD_CONTENT_TYPE + "'";
+
+ Map<String, String> errorMap = new HashMap<>();
+ errorMap.put("error", errorMessage);
+
+ String jsonResponse = JsonUtil.toJsonString(errorMap);
+
+ PrintWriter writer = httpRes.getWriter();
+ writer.write(jsonResponse);
+ writer.flush();
+ writer.close();
}
}
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
