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
commit d6dd5d59ee36331841dc6ba4d17f9746c14d168a Author: Gautier DI FOLCO <[email protected]> AuthorDate: Tue Feb 26 12:03:37 2019 +0100 JAMES-2670 Clean mailbox provisioning logging --- .../java/org/apache/james/imap/processor/AbstractAuthProcessor.java | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/protocols/imap/src/main/java/org/apache/james/imap/processor/AbstractAuthProcessor.java b/protocols/imap/src/main/java/org/apache/james/imap/processor/AbstractAuthProcessor.java index 1b7f59a..459f9d9 100644 --- a/protocols/imap/src/main/java/org/apache/james/imap/processor/AbstractAuthProcessor.java +++ b/protocols/imap/src/main/java/org/apache/james/imap/processor/AbstractAuthProcessor.java @@ -39,6 +39,7 @@ import org.apache.james.mailbox.model.MailboxConstants; import org.apache.james.mailbox.model.MailboxId; import org.apache.james.mailbox.model.MailboxPath; import org.apache.james.metrics.api.MetricFactory; +import org.apache.james.util.OptionalUtils; import org.slf4j.Logger; import org.slf4j.LoggerFactory; @@ -129,7 +130,8 @@ public abstract class AbstractAuthProcessor<M extends ImapRequest> extends Abstr } else { try { Optional<MailboxId> mailboxId = mailboxManager.createMailbox(inboxPath, mailboxSession); - LOGGER.info("Provisioning INBOX. {} created.", mailboxId); + OptionalUtils.executeIfEmpty(mailboxId, () -> LOGGER.warn("Provisioning INBOX successful. But no MailboxId have been returned.")) + .ifPresent(id -> LOGGER.info("Provisioning INBOX. {} created.", id)); } catch (MailboxExistsException e) { LOGGER.warn("Mailbox INBOX created by concurrent call. Safe to ignore this exception."); } --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
