print out tokens in IMAP

[org.ibex.mail.git] / src / org / ibex / mail / Address.java

diff --git a/src/org/ibex/mail/Address.java b/src/org/ibex/mail/Address.java
index 1bf6b04..100cf53 100644 (file)
--- a/src/org/ibex/mail/Address.java
+++ b/src/org/ibex/mail/Address.java
@@ -19,36 +19,55 @@ public class Address extends JSReflection implements Serializable {
     public static Address parse(String s) { try { return s==null?null:new Address(s); } catch (Malformed _) { return null; } }
     public Address(String user, String host, String description) {this.user=user;this.host=host;this.description=description;}
     public Address(String s0) throws Address.Malformed {
-        String s = s0.trim();
-        if (s.indexOf('<') == -1) {
-            if (s.indexOf(' ') == -1) {
-                description = "";
+        boolean good = false;
+        try {
+            String s = s0.trim();
+            if (s.indexOf('<') == -1) {
+                if (s.indexOf(' ') == -1) {
+                    description = "";
+                } else {
+                    description = s.substring(s.indexOf(' ')).trim();
+                    s = s.substring(0, s.indexOf(' '));
+                }
             } else {
-                description = s.substring(s.indexOf(' ')).trim();
-                s = s.substring(0, s.indexOf(' '));
+                if (s.indexOf('>') == -1) { throw new Malformed("found open-angle-bracket (<) but not close-angle-bracket (>)"); }
+                description = s.substring(0, s.indexOf('<')) + s.substring(s.indexOf('>') + 1);
+                s = s.substring(s.indexOf('<') + 1, s.indexOf('>'));
             }
-        } else {
-            if (s.indexOf('>') == -1) { throw new Malformed("found open-angle-bracket (<) but not close-angle-bracket (>)"); }
-            description = s.substring(0, s.indexOf('<')) + s.substring(s.indexOf('>') + 1);
-            s = s.substring(s.indexOf('<') + 1, s.indexOf('>'));
+            if (s.trim().length() == 0) {
+                user = null;
+                host = null;
+                good = true;
+            } else if (s.indexOf('@') == -1) { throw new Malformed("no @-sign in email address \""+s0+"\"");
+            } else {
+                user = s.substring(0, s.indexOf('@'));
+                host = s.substring(s.indexOf('@')+1);
+                good = true;
+            }
+        } finally {
+            if (!good) Log.error(Address.class, "problem parsing: \""+s0+"\"");
         }
-        if (s.indexOf('@') == -1) { throw new Malformed("no @-sign in email address \""+s0+"\""); }
-        user = s.substring(0, s.indexOf('@'));
-        host = s.substring(s.indexOf('@')+1);
     }
-    public String toString() { return user + "@" + host; }
+    public String toString() { return (user == null && host == null) ? "" : (user + "@" + host); }
     public String coerceToString() { return toString(); }
     public String toString(boolean desc) {
         return desc && description != null && description.length() > 0 ? (description+" <" + toString() + ">") : toString(); }
     public static class Malformed extends Message.Malformed { public Malformed(String s) { super(s); } }
 
+    /** in case of flaky DNS */
+    private static HashSet<String> local = new HashSet<String>();
     public boolean isLocal() {
-        InetAddress[] mx = SMTP.getMailExchangerIPs(host);
-        for(int i=0; i<mx.length; i++) {
-            try { if (NetworkInterface.getByInetAddress(mx[i]) != null) { return true; } }
-            catch (Exception e) { /* DELIBERATE */ }
+        synchronized(local) {
+            if (local.contains(host)) return true;
+            InetAddress[] mx = DNSUtil.getMailExchangerIPs(host);
+            for(int i=0; i<mx.length; i++) {
+                try { if (NetworkInterface.getByInetAddress(mx[i]) != null) {
+                    local.add(host);
+                    return true;
+                } } catch (Exception e) { /* DELIBERATE */ }
+            }
         }
-        Log.warn(this, "returning false");
+        Log.warn(this, "returning false for " + this + ".isLocal()");
         return false;
     }