This is an automated email from the ASF dual-hosted git repository.
btellier 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 1f08a8eafe [IMPROVMENT] JMAP upload/download with more context (#2252)
1f08a8eafe is described below
commit 1f08a8eafee1d2c766228118b543995e1119e2af
Author: Benoit TELLIER <[email protected]>
AuthorDate: Mon May 27 09:00:03 2024 +0200
[IMPROVMENT] JMAP upload/download with more context (#2252)
---
.../src/main/scala/org/apache/james/jmap/routes/DownloadRoutes.scala | 2 +-
.../src/main/scala/org/apache/james/jmap/routes/UploadRoutes.scala | 2 +-
2 files changed, 2 insertions(+), 2 deletions(-)
diff --git
a/server/protocols/jmap-rfc-8621/src/main/scala/org/apache/james/jmap/routes/DownloadRoutes.scala
b/server/protocols/jmap-rfc-8621/src/main/scala/org/apache/james/jmap/routes/DownloadRoutes.scala
index fb40769369..ff3074d629 100644
---
a/server/protocols/jmap-rfc-8621/src/main/scala/org/apache/james/jmap/routes/DownloadRoutes.scala
+++
b/server/protocols/jmap-rfc-8621/src/main/scala/org/apache/james/jmap/routes/DownloadRoutes.scala
@@ -284,7 +284,7 @@ class DownloadRoutes
@Inject()(@Named(InjectionKeys.RFC_8621) val authenticator:
ProblemDetails(status = NOT_FOUND, detail = "The resource could
not be found"),
NOT_FOUND)
case e =>
- LOGGER.error("Unexpected error upon downloads", e)
+ LOGGER.error("Unexpected error upon download {}", request.uri(), e)
respondDetails(response,
ProblemDetails(status = INTERNAL_SERVER_ERROR, detail =
e.getMessage),
INTERNAL_SERVER_ERROR)
diff --git
a/server/protocols/jmap-rfc-8621/src/main/scala/org/apache/james/jmap/routes/UploadRoutes.scala
b/server/protocols/jmap-rfc-8621/src/main/scala/org/apache/james/jmap/routes/UploadRoutes.scala
index 8b6e038f1c..b80d6ce2a4 100644
---
a/server/protocols/jmap-rfc-8621/src/main/scala/org/apache/james/jmap/routes/UploadRoutes.scala
+++
b/server/protocols/jmap-rfc-8621/src/main/scala/org/apache/james/jmap/routes/UploadRoutes.scala
@@ -103,7 +103,7 @@ class UploadRoutes @Inject()(@Named(InjectionKeys.RFC_8621)
val authenticator: A
ProblemDetails(status = FORBIDDEN, detail = "Upload to other
accounts is forbidden"),
FORBIDDEN)
case e =>
- LOGGER.error("Unexpected error upon uploads", e)
+ LOGGER.error("Unexpected error upon upload {}", request.uri(), e)
respondDetails(response,
ProblemDetails(status = INTERNAL_SERVER_ERROR, detail =
e.getMessage),
INTERNAL_SERVER_ERROR)
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
