JAMES-2530 GetMailboxes should use more fluent wrapAround MDC capability

Project: http://git-wip-us.apache.org/repos/asf/james-project/repo
Commit: http://git-wip-us.apache.org/repos/asf/james-project/commit/80d0dec7
Tree: http://git-wip-us.apache.org/repos/asf/james-project/tree/80d0dec7
Diff: http://git-wip-us.apache.org/repos/asf/james-project/diff/80d0dec7

Branch: refs/heads/master
Commit: 80d0dec7c19e66ec895212b09641f3e84577c76a
Parents: 7214635
Author: Benoit Tellier <btell...@linagora.com>
Authored: Tue Aug 28 10:22:47 2018 +0700
Committer: Antoine Duprat <adup...@linagora.com>
Committed: Tue Aug 28 14:11:52 2018 +0200

----------------------------------------------------------------------
 .../apache/james/jmap/methods/GetMailboxesMethod.java  | 13 ++++++-------
 1 file changed, 6 insertions(+), 7 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/james-project/blob/80d0dec7/server/protocols/jmap/src/main/java/org/apache/james/jmap/methods/GetMailboxesMethod.java
----------------------------------------------------------------------
diff --git 
a/server/protocols/jmap/src/main/java/org/apache/james/jmap/methods/GetMailboxesMethod.java
 
b/server/protocols/jmap/src/main/java/org/apache/james/jmap/methods/GetMailboxesMethod.java
index c27ff9b..62c827d 100644
--- 
a/server/protocols/jmap/src/main/java/org/apache/james/jmap/methods/GetMailboxesMethod.java
+++ 
b/server/protocols/jmap/src/main/java/org/apache/james/jmap/methods/GetMailboxesMethod.java
@@ -81,13 +81,12 @@ public class GetMailboxesMethod implements Method {
         Preconditions.checkArgument(request instanceof GetMailboxesRequest);
         GetMailboxesRequest mailboxesRequest = (GetMailboxesRequest) request;
         return metricFactory.withMetric(JMAP_PREFIX + METHOD_NAME.getName(),
-            () -> MDCBuilder.withMdc(
-                MDCBuilder.create()
-                    .addContext(MDCBuilder.ACTION, "GET_MAILBOXES")
-                    .addContext("accountId", mailboxesRequest.getAccountId())
-                    .addContext("mailboxIds", mailboxesRequest.getIds())
-                    .addContext("properties", 
mailboxesRequest.getProperties()),
-                () -> process(clientId, mailboxSession, mailboxesRequest)));
+            MDCBuilder.create()
+                .addContext(MDCBuilder.ACTION, "GET_MAILBOXES")
+                .addContext("accountId", mailboxesRequest.getAccountId())
+                .addContext("mailboxIds", mailboxesRequest.getIds())
+                .addContext("properties", mailboxesRequest.getProperties())
+                .wrapArround(() -> process(clientId, mailboxSession, 
mailboxesRequest)));
     }
 
     private Stream<JmapResponse> process(ClientId clientId, MailboxSession 
mailboxSession, GetMailboxesRequest mailboxesRequest) {


---------------------------------------------------------------------
To unsubscribe, e-mail: server-dev-unsubscr...@james.apache.org
For additional commands, e-mail: server-dev-h...@james.apache.org

Reply via email to