Author: norman Date: Fri Oct 7 16:36:25 2011 New Revision: 1180095 URL: http://svn.apache.org/viewvc?rev=1180095&view=rev Log: First add the IdleStateListener and then write the response back the client. See IMAP-341
Modified: james/imap/trunk/processor/src/main/java/org/apache/james/imap/processor/IdleProcessor.java Modified: james/imap/trunk/processor/src/main/java/org/apache/james/imap/processor/IdleProcessor.java URL: http://svn.apache.org/viewvc/james/imap/trunk/processor/src/main/java/org/apache/james/imap/processor/IdleProcessor.java?rev=1180095&r1=1180094&r2=1180095&view=diff ============================================================================== --- james/imap/trunk/processor/src/main/java/org/apache/james/imap/processor/IdleProcessor.java (original) +++ james/imap/trunk/processor/src/main/java/org/apache/james/imap/processor/IdleProcessor.java Fri Oct 7 16:36:25 2011 @@ -74,9 +74,7 @@ public class IdleProcessor extends Abstr protected void doProcess(final IdleRequest message, final ImapSession session, final String tag, final ImapCommand command, final Responder responder) { try { - responder.respond(new ContinuationResponse(HumanReadableText.IDLING)); - unsolicitedResponses(session, responder, false); - + final MailboxManager mailboxManager = getMailboxManager(); final MailboxSession mailboxSession = ImapSessionUtils.getMailboxSession(session); final SelectedMailbox sm = session.getSelected(); @@ -148,6 +146,12 @@ public class IdleProcessor extends Abstr } }, heartbeatInterval, heartbeatIntervalUnit); } + + // Write the response after the listener was add + // IMAP-341 + responder.respond(new ContinuationResponse(HumanReadableText.IDLING)); + unsolicitedResponses(session, responder, false); + } catch (MailboxException e) { session.getLog().debug("Idle failed", e); --------------------------------------------------------------------- To unsubscribe, e-mail: server-dev-unsubscr...@james.apache.org For additional commands, e-mail: server-dev-h...@james.apache.org