Now that was a nasty bug. The Javamail API changed. Hmm... now we have two working news stores and maybe even two working POP3 stores.

grendel/storage/NewsStore.java

@@ -128,6 +128,7 @@ public class NewsStore extends Store {
 
 
   protected boolean protocolConnect(String host,
+                                    int port,
                                     String user,
                                     String password)
     throws MessagingException {
@@ -152,8 +153,6 @@ public class NewsStore extends Store {
 
     nntp = new NNTPConnection();
 
-    int port = -1;     // #### override this from the url or something.
-
     try {
       boolean status = nntp.connect(host, port, user, password);
       if (!status) return false;

grendel/storage/PopStore.java

@@ -103,6 +103,7 @@ public class PopStore extends Store {
   }
 
   synchronized public boolean protocolConnect(String host,
+                                              int port,
                                               String user,
                                               String password)
     throws MessagingException
@@ -110,6 +111,7 @@ public class PopStore extends Store {
 
     if (fSocket != null) return true;   // Already connected.
     try {
+      if (port != -1) fPort = port;
       fSocket = new Socket(host, fPort);
       fInput = fSocket.getInputStream();
       fOutput = new DataOutputStream(fSocket.getOutputStream());