TOP handler is broken

[Brian]

The TOP handler is passing the message number and line count to the
function to get the message headers, and throwing an exception that
hangs the server.

I wrote a simple fix that splits off the message ID and ignores the
line count:


Index: src/web-mail/web-mail/content/web-mail/server/
popConnectionHandler.js
===================================================================
RCS file: /cvs/webmail/src/web-mail/web-mail/content/web-mail/server/
popConnectionHandler.js,v
retrieving revision 1.15
diff -u -b -B -r1.15 popConnectionHandler.js
--- src/web-mail/web-mail/content/web-mail/server/
popConnectionHandler.js 5 Mar 2009 18:26:05 -0000       1.15
+++ src/web-mail/web-mail/content/web-mail/server/
popConnectionHandler.js 31 Jul 2010 23:55:06 -0000
@@ -248,7 +248,9 @@
                     try
                     {
                         this.m_POPLog.Write("POPconnectionHandler -
onDataWritable - top -" + szParam );
-                        if (!
this.m_DomainHandler.getMessageHeaders(szParam))
+                       var iId = szParam.split(" ")[0];
+
+                        if (!
this.m_DomainHandler.getMessageHeaders(iId))
                             throw new Error("TOP NOT supported");
                     }
                     catch(e)