Author: norman
Date: Mon Jan 17 11:14:00 2011
New Revision: 1059876

URL: http://svn.apache.org/viewvc?rev=1059876&view=rev
Log:
respond with UNKNOWN_COMMAND if compress is not supported

Modified:
    
james/imap/trunk/processor/src/main/java/org/apache/james/imap/processor/CompressProcessor.java

Modified: 
james/imap/trunk/processor/src/main/java/org/apache/james/imap/processor/CompressProcessor.java
URL: 
http://svn.apache.org/viewvc/james/imap/trunk/processor/src/main/java/org/apache/james/imap/processor/CompressProcessor.java?rev=1059876&r1=1059875&r2=1059876&view=diff
==============================================================================
--- 
james/imap/trunk/processor/src/main/java/org/apache/james/imap/processor/CompressProcessor.java
 (original)
+++ 
james/imap/trunk/processor/src/main/java/org/apache/james/imap/processor/CompressProcessor.java
 Mon Jan 17 11:14:00 2011
@@ -58,23 +58,22 @@ public class CompressProcessor extends A
     protected void doProcess(ImapMessage acceptableMessage, Responder 
responder, ImapSession session) {
         CompressRequest request = (CompressRequest) acceptableMessage;
         if (session.isCompressionSupported()) {
-            boolean compressed = (Boolean) session.getAttribute(COMPRESSED);
-            if (compressed) {
+            Object obj = session.getAttribute(COMPRESSED);
+            if (obj != null) {
                 responder.respond(factory.taggedNo(request.getTag(), 
request.getCommand(), HumanReadableText.COMPRESS_ALREADY_ACTIVE));
             } else {
                 if (request.getAlgorithm().equalsIgnoreCase(ALGO) == false) {
                     responder.respond(factory.taggedBad(request.getTag(), 
request.getCommand(), HumanReadableText.ILLEGAL_ARGUMENTS));
                 } else {
+                    responder.respond(factory.taggedOk(request.getTag(), 
request.getCommand(), HumanReadableText.DEFLATE_ACTIVE));
+
                     if (session.startCompression()) {
                         session.setAttribute(COMPRESSED, true);
-                        responder.respond(factory.taggedOk(request.getTag(), 
request.getCommand(), HumanReadableText.DEFLATE_ACTIVE));
-                    } else {
-                        
                     }
                 }
             }
         } else {
-            responder.respond(factory.taggedBad(request.getTag(), 
request.getCommand(), HumanReadableText.INVALID_COMMAND));
+            responder.respond(factory.taggedBad(request.getTag(), 
request.getCommand(), HumanReadableText.UNKNOWN_COMMAND));
         }
     }
 



---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to