+// Copyright 2000-2005 the Contributors, as shown in the revision logs.
+// Licensed under the Apache Public Source License 2.0 ("the License").
+// You may not use this file except in compliance with the License.
+
package org.ibex.mail.protocol;
import org.ibex.mail.*;
import org.ibex.mail.target.*;
-import org.ibex.jinetd.Worker;
import org.ibex.util.*;
+import org.ibex.net.*;
import org.ibex.io.*;
import java.net.*;
import java.io.*;
import javax.naming.*;
import javax.naming.directory.*;
+// FIXME: better delivery cycle attempt algorithm; current one sucks
+// FIXME: logging: current logging sucks
+// FIXME: loop prevention
+
+// graylisting?
+
+// FEATURE: infer messageid, date, if not present (?)
// FEATURE: exponential backoff on retry time?
+// FEATURE: RFC2822, section 4.5.1: special "postmaster" address
+// FEATURE: RFC2822, section 4.5.4.1: retry strategies
+// FEATURE: RFC2822, section 5, multiple MX records, preferences, ordering
+// FEATURE: RFC2822, end of 4.1.2: backslashes in headers
public class SMTP {
+ public static final SimpleDateFormat dateFormat = new SimpleDateFormat("EEE, d MMM yyyy HH:mm:ss Z");
+ public static final int numOutgoingThreads = 5;
private static final Mailbox spool =
FileBasedMailbox.getFileBasedMailbox(Mailbox.STORAGE_ROOT,false).slash("spool",true).slash("smtp",true);
- static { new Thread() { public void run() { Outgoing.runq(); } }.start(); }
+ static {
+ for(int i=0; i<numOutgoingThreads; i++)
+ new Outgoing().start();
+ }
+
+ public static void accept(Message m) throws IOException {
+ if (!m.envelopeTo.isLocal()) Outgoing.accept(m);
+ else Target.root.accept(m);
+ }
+
+ public static class SMTPException extends MailException {
+ int code;
+ String message;
+ public SMTPException(String s) {
+ try {
+ code = Integer.parseInt(s.substring(0, s.indexOf(' ')));
+ message = s.substring(s.indexOf(' ')+1);
+ } catch (NumberFormatException nfe) {
+ code = -1;
+ message = s;
+ }
+ }
+ public String toString() { return "SMTP " + code + ": " + message; }
+ public String getMessage() { return toString(); }
+ }
// Server //////////////////////////////////////////////////////////////////////////////
- public static class Server implements Worker {
+ public static class Server {
public void handleRequest(Connection conn) {
- Log.error(this, "accepted...");
conn.setTimeout(5 * 60 * 1000);
+ conn.setNewline("\r\n");
conn.println("220 " + conn.vhost + " SMTP " + this.getClass().getName());
Address from = null;
Vector to = new Vector();
+ boolean ehlo = false;
+ String remotehost = null;
for(String command = conn.readln(); ; command = conn.readln()) try {
if (command == null) return;
+ Log.warn(conn.getRemoteAddress()+"", command);
String c = command.toUpperCase();
- if (c.startsWith("HELO")) { conn.println("250 HELO " + conn.vhost); from = null; to = new Vector();
- } else if (c.startsWith("EHLO")) { conn.println("250"); from = null; to = new Vector();
+ if (c.startsWith("HELO")) {
+ remotehost = c.substring(5).trim();
+ conn.println("250 HELO " + conn.vhost);
+ from = null; to = new Vector();
+ } else if (c.startsWith("EHLO")) {
+ remotehost = c.substring(5).trim();
+ conn.println("250 "+conn.vhost+" greets " + remotehost);
+ ehlo = true;
+ from = null; to = new Vector();
} else if (c.startsWith("RSET")) { conn.println("250 reset ok"); from = null; to = new Vector();
} else if (c.startsWith("HELP")) { conn.println("214 you are beyond help. see a trained professional.");
- } else if (c.startsWith("VRFY")) { conn.println("252 We don't VRFY; proceed anyway");
- } else if (c.startsWith("EXPN")) { conn.println("550 EXPN not available");
+ } else if (c.startsWith("VRFY")) { conn.println("502 VRFY not supported");
+ } else if (c.startsWith("EXPN")) { conn.println("502 EXPN not supported");
} else if (c.startsWith("NOOP")) { conn.println("250 OK");
} else if (c.startsWith("QUIT")) { conn.println("221 " + conn.vhost + " closing connection"); return;
} else if (c.startsWith("MAIL FROM:")) {
- conn.println("250 " + (from = new Address(command.substring(10).trim())) + " is syntactically correct");
+ command = command.substring(10).trim();
+ from = command.equals("<>") ? null : new Address(command);
+ conn.println("250 " + from + " is syntactically correct");
} else if (c.startsWith("RCPT TO:")) {
- if (from == null) { conn.println("503 MAIL FROM must precede RCPT TO"); continue; }
+ // some clients are broken and put RCPT first; we will tolerate this
command = command.substring(8).trim();
if(command.indexOf(' ') != -1) command = command.substring(0, command.indexOf(' '));
Address addr = new Address(command);
- if (addr.isLocal()) conn.println("250 " + addr + " is on this machine; I will deliver it");
- else if (conn.getRemoteAddress().isLoopbackAddress())
+ if (addr.isLocal()) {
+ // FEATURE: should check the address further and give 550 if undeliverable
+ conn.println("250 " + addr + " is on this machine; I will deliver it");
+ to.addElement(addr);
+ } else if (conn.getRemoteAddress().isLoopbackAddress()) {
conn.println("250 you are connected locally, so I will let you send");
- else { conn.println("551 sorry, " + addr + " is not on this machine"); }
- to.addElement(addr);
+ to.addElement(addr);
+ } else {
+ conn.println("551 sorry, " + addr + " is not on this machine");
+ }
} else if (c.startsWith("DATA")) {
- if (from == null) { conn.println("503 MAIL FROM command must precede DATA"); continue; }
- if (to == null) { conn.println("503 RCPT TO command must precede DATA"); continue; }
+ //if (from == null) { conn.println("503 MAIL FROM command must precede DATA"); continue; }
+ if (to == null || to.size()==0) { conn.println("503 RCPT TO command must precede DATA"); continue; }
conn.println("354 Enter message, ending with \".\" on a line by itself");
conn.flush();
try {
StringBuffer buf = new StringBuffer();
+ buf.append("Received: from " + conn.getRemoteHostname() + " (" + remotehost + ")\r\n");
+ buf.append(" by "+conn.vhost+" ("+SMTP.class.getName()+") with "+(ehlo?"ESMTP":"SMTP") + "\r\n");
+ buf.append(" for ");
+ // FIXME: this is leaking BCC addrs
+ // for(int i=0; i<to.size(); i++) buf.append(to.elementAt(i) + " ");
+ buf.append("; " + dateFormat.format(new Date()) + "\r\n");
while(true) {
String s = conn.readln();
if (s == null) throw new RuntimeException("connection closed");
String body = buf.toString();
Message m = null;
for(int i=0; i<to.size(); i++) {
- m = new Message(from, (Address)to.elementAt(i), new Stream(body));
- if (!m.envelopeTo.isLocal()) Outgoing.accept(m);
- else Target.root.accept(m);
+ m = Message.newMessage(new Fountain.StringFountain(body), from, (Address)to.elementAt(i));
+ accept(m);
}
if (m != null) Log.info(SMTP.class, "accepted message: " + m.summary());
conn.println("250 message accepted");
from = null; to = new Vector();
} catch (MailException.Malformed mfe) { conn.println("501 " + mfe.toString());
} catch (MailException.MailboxFull mbf) { conn.println("452 " + mbf);
- } catch (IOException ioe) { conn.println("554 " + ioe.toString());
+ } catch (Later.LaterException le) { conn.println("453 try again later");
+ } catch (IOException ioe) {
+ //conn.println("554 " + ioe.toString());
+ Log.error(this, ioe);
+ conn.close();
+ return;
}
- } else { conn.println("500 unrecognized command"); }
- } catch (Message.Malformed e) { conn.println("501 " + e.toString()); /* FIXME could be wrong code */ }
+ } else { conn.println("500 unrecognized command"); }
+ } catch (Message.Malformed e) { conn.println("501 " + e.toString()); }
}
}
// Outgoing Mail Thread //////////////////////////////////////////////////////////////////////////////
- public static class Outgoing {
+ public static class Outgoing extends Thread {
private static final HashSet deadHosts = new HashSet();
public static void accept(Message m) throws IOException {
- Log.info(SMTP.class, "queued:\n" + m.summary());
- if (m.traces.length >= 100)
- Log.warn(SMTP.Outgoing.class, "Message with " + m.traces.length + " trace hops; dropping\n" + m.summary());
- else synchronized(Outgoing.class) {
+ if (m == null) { Log.warn(Outgoing.class, "attempted to accept(null)"); return; }
+ String traces = m.headers.get("Received");
+ if (traces!=null) {
+ int lines = 0;
+ for(int i=0; i<traces.length(); i++)
+ if (traces.charAt(i)=='\n' || traces.charAt(i)=='\r')
+ lines++;
+ if (lines > 100) { // required by rfc
+ Log.warn(SMTP.Outgoing.class, "Message with " + lines + " trace hops; dropping\n" + m.summary());
+ return;
+ }
+ }
+ synchronized(Outgoing.class) {
spool.add(m);
- Outgoing.class.notify();
+ Outgoing.class.notifyAll();
}
}
- public static boolean attempt(Message m) throws IOException {
+ public static boolean attempt(Message m) throws IOException { return attempt(m, false); }
+ public static boolean attempt(Message m, boolean noBounces) throws IOException {
+ if (m.envelopeTo == null) {
+ Log.warn(SMTP.Outgoing.class, "aieeee, null envelopeTo: " + m.summary());
+ return false;
+ }
InetAddress[] mx = getMailExchangerIPs(m.envelopeTo.host);
if (mx.length == 0) {
- Log.warn(SMTP.Outgoing.class, "could not resolve " + m.envelopeTo.host + "; bouncing it\n" + m.summary());
- accept(m.bounce("could not resolve " + m.envelopeTo.host));
- return true;
+ if (!noBounces) {
+ accept(m.bounce("could not resolve " + m.envelopeTo.host));
+ return true;
+ } else {
+ Log.warn(SMTP.Outgoing.class, "could not resolve " + m.envelopeTo.host);
+ return false;
+ }
}
if (new Date().getTime() - m.arrival.getTime() > 1000 * 60 * 60 * 24 * 5) {
- Log.warn(SMTP.Outgoing.class, "could not send message after 5 days; bouncing it\n" + m.summary());
- accept(m.bounce("could not send for 5 days"));
- return true;
+ if (!noBounces) {
+ accept(m.bounce("could not send for 5 days"));
+ return true;
+ } else {
+ Log.warn(SMTP.Outgoing.class, "could not send for 5 days: " + m.summary());
+ return false;
+ }
}
for(int i=0; i<mx.length; i++) {
- if (deadHosts.contains(mx[i])) continue;
+ //if (deadHosts.contains(mx[i])) continue;
if (attempt(m, mx[i])) { return true; }
}
return false;
}
private static void check(String s, Connection conn) {
- while (s.charAt(3) == '-') s = conn.readln();
- if (s.startsWith("4")||s.startsWith("5")) throw new MailException(s);
+ if (s==null) return;
+ while (s.length() > 3 && s.charAt(3) == '-') s = conn.readln();
+ if (s.startsWith("4")||s.startsWith("5")) throw new SMTPException(s);
}
private static boolean attempt(final Message m, final InetAddress mx) {
boolean accepted = false;
Connection conn = null;
try {
- Log.info(SMTP.Outgoing.class, "connecting to " + mx + "...");
+ Log.note("connecting to " + mx + "...");
conn = new Connection(new Socket(mx, 25), InetAddress.getLocalHost().getHostName());
+ conn.setNewline("\r\n");
conn.setTimeout(60 * 1000);
- Log.info(SMTP.Outgoing.class, "connected");
+ Log.note(" connected");
check(conn.readln(), conn); // banner
- conn.println("HELO " + conn.vhost); check(conn.readln(), conn);
- conn.println("MAIL FROM:<" + m.envelopeFrom.user + "@" + m.envelopeFrom.host+">"); check(conn.readln(), conn);
- conn.println("RCPT TO:<" + m.envelopeTo.user + "@" + m.envelopeTo.host+">"); check(conn.readln(), conn);
+ try {
+ conn.println("EHLO " + conn.vhost);
+ check(conn.readln(), conn);
+ } catch (SMTPException smtpe) {
+ conn.println("HELO " + conn.vhost);
+ check(conn.readln(), conn);
+ }
+ if (m.envelopeFrom==null) {
+ Log.warn("", "MAIL FROM:<>");
+ conn.println("MAIL FROM:<>"); check(conn.readln(), conn);
+ } else {
+ Log.warn("", "MAIL FROM:<" + m.envelopeFrom.toString()+">");
+ conn.println("MAIL FROM:<" + m.envelopeFrom.toString()+">"); check(conn.readln(), conn);
+ }
+ conn.println("RCPT TO:<" + m.envelopeTo.toString()+">"); check(conn.readln(), conn);
conn.println("DATA"); check(conn.readln(), conn);
- conn.println(m.toString());
+ Headers head = m.headers;
+ head.remove("return-path");
+ head.remove("bcc");
+ Stream stream = head.getStream();
+ for(String s = stream.readln(); s!=null; s=stream.readln()) {
+ if (s.startsWith(".")) conn.print(".");
+ conn.println(s);
+ }
+ conn.println("");
+ stream = m.getBody().getStream();
+ for(String s = stream.readln(); s!=null; s=stream.readln()) {
+ if (s.startsWith(".")) conn.print(".");
+ conn.println(s);
+ }
conn.println(".");
- check(conn.readln(), conn);
- Log.info(SMTP.Outgoing.class, "success: message accepted by " + mx);
+ String resp = conn.readln();
+ if (resp == null)
+ throw new SMTPException("server " + mx + " closed connection without accepting message");
+ check(resp, conn);
+ Log.warn(SMTP.Outgoing.class, "success: " + mx + " accepted " + m.summary() + "\n["+resp+"]");
accepted = true;
conn.close();
+ } catch (SMTPException e) {
+ if (accepted) return true;
+ Log.warn(SMTP.Outgoing.class, " unable to send; error=" + e);
+ Log.warn(SMTP.Outgoing.class, " message: " + m.summary());
+ Log.warn(SMTP.Outgoing.class, e);
+ if (e.code >= 500 && e.code <= 599) {
+ try {
+ attempt(m.bounce("unable to deliver: " + e), true);
+ } catch (Exception ex) {
+ Log.error(SMTP.Outgoing.class, "exception while trying to deliver bounce; giving up completely");
+ Log.error(SMTP.Outgoing.class, ex);
+ }
+ return true;
+ }
+ return false;
} catch (Exception e) {
if (accepted) return true;
- Log.warn(SMTP.Outgoing.class, "unable to send; error=" + e);
+ Log.warn(SMTP.Outgoing.class, " unable to send; error=" + e);
+ Log.warn(SMTP.Outgoing.class, " message: " + m.summary());
Log.warn(SMTP.Outgoing.class, e);
+ if (conn != null) Log.warn(SMTP.Outgoing.class, conn.dumpLog());
return false;
} finally {
if (conn != null) conn.close();
return accepted;
}
- static void runq() {
+ private static HashSet<Outgoing> threads = new HashSet<Outgoing>();
+ private static int serials = 1;
+ private int serial = serials++;
+ private Mailbox.Iterator it;
+
+ public Outgoing() {
+ synchronized(Outgoing.class) {
+ threads.add(this);
+ }
+ }
+
+ public void wake() {
+ int count = spool.count(Query.all());
+ Log.info(SMTP.Outgoing.class, "outgoing thread #"+serial+" woke up; " + count + " messages to send");
try {
- Log.setThreadAnnotation("[outgoing smtp] ");
- Log.info(SMTP.Outgoing.class, "outgoing thread started; " + spool.count(Query.all()) + " messages to send");
while(true) {
- for(Mailbox.Iterator it = spool.iterator(); it.next(); ) {
- try { if (attempt(it.cur())) it.delete(); }
- catch (Exception e) { Log.error(SMTP.Outgoing.class, e); }
+ boolean good = false;
+ synchronized(Outgoing.class) {
+ it = spool.iterator();
+ OUTER: for(; it.next(); ) {
+ for(Outgoing o : threads)
+ if (o!=this && o.it != null && o.it.uid()==it.uid())
+ continue OUTER;
+ good = true;
+ break;
+ }
}
+ if (!good) break;
+ try {
+ if (attempt(it.cur())) it.delete();
+ } catch (Exception e) {
+ Log.error(SMTP.Outgoing.class, e);
+ }
+ Log.info(this, "sleeping for 3s...");
+ Thread.sleep(3000);
+ }
+ } catch (Exception e) {
+ //if (e instanceof InterruptedException) throw e;
+ Log.error(SMTP.Outgoing.class, e);
+ }
+ Log.info(SMTP.Outgoing.class, "outgoing thread #"+serial+" going back to sleep");
+ it = null;
+ }
+
+ public void run() {
+ try {
+ while(true) {
+ Log.setThreadAnnotation("[outgoing #"+serial+"] ");
+ wake();
+ Thread.sleep(1000);
synchronized(Outgoing.class) {
- Log.info(SMTP.Outgoing.class, "outgoing thread going to sleep");
Outgoing.class.wait(5 * 60 * 1000);
- deadHosts.clear();
- Log.info(SMTP.Outgoing.class,"outgoing thread woke up; "+spool.count(Query.all())+" messages in queue");
}
}
- } catch (Exception e) { Log.error(SMTP.Outgoing.class, e); }
+ } catch (InterruptedException e) { Log.warn(this, e); }
}
}
} else {
ret = new InetAddress[attr.size()];
NamingEnumeration ne = attr.getAll();
- for(int i=0; ne.hasMore(); i++) {
+ for(int i=0; ne.hasMore();) {
String mx = (String)ne.next();
// FIXME we should be sorting here
mx = mx.substring(mx.indexOf(" ") + 1);
if (mx.charAt(mx.length() - 1) == '.') mx = mx.substring(0, mx.length() - 1);
- ret[i] = InetAddress.getByName(mx);
+ InetAddress ia = InetAddress.getByName(mx);
+ if (ia.equals(IP.getIP(127,0,0,1))) continue;
+ ret[i++] = ia;
}
}
} catch (Exception e) {