This is an automated email from the ASF dual-hosted git repository. rcordier pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/james-project.git
The following commit(s) were added to refs/heads/master by this push: new f0abcc2898 [REFACTOR] Remove unused JMAP code (#2592) f0abcc2898 is described below commit f0abcc289842103be8d3f5a6c02f8399defcc244 Author: Benoit TELLIER <btell...@linagora.com> AuthorDate: Tue Jan 14 04:34:29 2025 +0100 [REFACTOR] Remove unused JMAP code (#2592) --- .../java/org/apache/james/jmap/HttpConstants.java | 1 - .../java/org/apache/james/jmap/JMAPRoutes.java | 23 ---------------------- .../main/java/org/apache/james/jmap/JMAPUrls.java | 3 --- 3 files changed, 27 deletions(-) diff --git a/server/protocols/jmap/src/main/java/org/apache/james/jmap/HttpConstants.java b/server/protocols/jmap/src/main/java/org/apache/james/jmap/HttpConstants.java index ce50efae7c..d5daa3d844 100644 --- a/server/protocols/jmap/src/main/java/org/apache/james/jmap/HttpConstants.java +++ b/server/protocols/jmap/src/main/java/org/apache/james/jmap/HttpConstants.java @@ -22,5 +22,4 @@ package org.apache.james.jmap; public interface HttpConstants { String JSON_CONTENT_TYPE = "application/json"; String JSON_CONTENT_TYPE_UTF8 = "application/json; charset=UTF-8"; - String TEXT_PLAIN_CONTENT_TYPE = "text/plain"; } diff --git a/server/protocols/jmap/src/main/java/org/apache/james/jmap/JMAPRoutes.java b/server/protocols/jmap/src/main/java/org/apache/james/jmap/JMAPRoutes.java index 7c54a5acd7..efd0861604 100644 --- a/server/protocols/jmap/src/main/java/org/apache/james/jmap/JMAPRoutes.java +++ b/server/protocols/jmap/src/main/java/org/apache/james/jmap/JMAPRoutes.java @@ -19,18 +19,10 @@ package org.apache.james.jmap; -import static io.netty.handler.codec.http.HttpResponseStatus.BAD_REQUEST; import static io.netty.handler.codec.http.HttpResponseStatus.FOUND; -import static io.netty.handler.codec.http.HttpResponseStatus.INTERNAL_SERVER_ERROR; -import static io.netty.handler.codec.http.HttpResponseStatus.UNAUTHORIZED; import java.util.stream.Stream; -import org.slf4j.Logger; - -import reactor.core.publisher.Mono; -import reactor.netty.http.server.HttpServerResponse; - public interface JMAPRoutes { Stream<JMAPRoute> routes(); @@ -47,19 +39,4 @@ public interface JMAPRoutes { static JMAPRoute.Action redirectTo(String location) { return (req, res) -> res.status(FOUND).header("Location", location).send(); } - - default Mono<Void> handleInternalError(HttpServerResponse response, Logger logger, Throwable e) { - logger.error("Internal server error", e); - return response.status(INTERNAL_SERVER_ERROR).send(); - } - - default Mono<Void> handleBadRequest(HttpServerResponse response, Logger logger, Throwable e) { - logger.warn("Invalid request received.", e); - return response.status(BAD_REQUEST).send(); - } - - default Mono<Void> handleAuthenticationFailure(HttpServerResponse response, Logger logger, Throwable e) { - logger.warn("Unauthorized", e); - return response.status(UNAUTHORIZED).send(); - } } diff --git a/server/protocols/jmap/src/main/java/org/apache/james/jmap/JMAPUrls.java b/server/protocols/jmap/src/main/java/org/apache/james/jmap/JMAPUrls.java index f409893a80..4d86b73863 100644 --- a/server/protocols/jmap/src/main/java/org/apache/james/jmap/JMAPUrls.java +++ b/server/protocols/jmap/src/main/java/org/apache/james/jmap/JMAPUrls.java @@ -23,8 +23,5 @@ public interface JMAPUrls { String JMAP = "/jmap"; String JMAP_WS = "/jmap/ws"; String AUTHENTICATION = "/authentication"; - String DOWNLOAD = "/download"; - String UPLOAD = "/upload"; String EVENT_SOURCE = "/eventSource"; - String NOT_IMPLEMENTED = "/notImplemented"; } --------------------------------------------------------------------- To unsubscribe, e-mail: notifications-unsubscr...@james.apache.org For additional commands, e-mail: notifications-h...@james.apache.org