import java.text.*;
// FIXME: inbound throttling/ratelimiting
+// FIXME: probably need some throttling on outbound mail
+// FEATURE: rate-limiting
// "Address enumeration detection" -- notice when it looks like somebody
// is trying a raft of addresses.
-// RFC's implemented
// RFC2554: SMTP Service Extension for Authentication
// - did not implement section 5, though
// RFC4616: SASL PLAIN
// FIXME: logging: current logging sucks
// FIXME: loop prevention
-// FIXME: probably need some throttling on outbound mail
-// FEATURE: public static boolean validate(Address a)
-// FEATURE: rate-limiting
+// FEATURE: infer date if not present
-// 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
+ Any system that includes an SMTP server supporting mail relaying or
+ delivery MUST support the reserved mailbox "postmaster" as a case-
+ insensitive local name. This postmaster address is not strictly
+ necessary if the server always returns 554 on connection opening (as
+ described in section 3.1). The requirement to accept mail for
+ postmaster implies that RCPT commands which specify a mailbox for
+ postmaster at any of the domains for which the SMTP server provides
+ mail service, as well as the special case of "RCPT TO:<Postmaster>"
+ (with no domain specification), MUST be supported.
+
+ */
// FEATURE: RFC2822, section 5, multiple MX records, preferences, ordering
// FEATURE: RFC2822, end of 4.1.2: backslashes in headers
+// FEATURE: batching retrys by host (retry multiple in one session, keep retry intervals on a host basis not a message basis)
+// FEATURE: first two attempts should be close together (rec'd by 2821)
+/*
+ // FEATURE: RFC2822, section 4.5.4.1: retry strategies
+ // per-command, per-attempt timeouts
+ Experience suggests that failures are typically transient (the target
+ system or its connection has crashed), favoring a policy of two
+ connection attempts in the first hour the message is in the queue,
+ and then backing off to one every two or three hours.
+
+ The SMTP client can shorten the queuing delay in cooperation with the
+ SMTP server. For example, if mail is received from a particular
+ address, it is likely that mail queued for that host can now be sent.
+ Application of this principle may, in many cases, eliminate the
+ requirement for an explicit "send queues now" function such as ETRN
+ [9].
+
+ An SMTP client may have a large queue of messages for each
+ unavailable destination host. If all of these messages were retried
+ in every retry cycle, there would be excessive Internet overhead and
+ the sending system would be blocked for a long period. Note that an
+ SMTP client can generally determine that a delivery attempt has
+ failed only after a timeout of several minutes and even a one-minute
+ timeout per connection will result in a very large delay if retries
+ are repeated for dozens, or even hundreds, of queued messages to the
+ same host.
+
+ When a mail message is to be delivered to multiple recipients, and
+ the SMTP server to which a copy of the message is to be sent is the
+ same for multiple recipients, then only one copy of the message
+ SHOULD be transmitted. That is, the SMTP client SHOULD use the
+ command sequence: MAIL, RCPT, RCPT,... RCPT, DATA instead of the
+ sequence: MAIL, RCPT, DATA, ..., MAIL, RCPT, DATA. However, if there
+ are very many addresses, a limit on the number of RCPT commands per
+ MAIL command MAY be imposed. Implementation of this efficiency
+ feature is strongly encouraged.
+ */
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 SqliteMailbox allmail =
(SqliteMailbox)FileBasedMailbox
- .getFileBasedMailbox("/afs/megacz.com/mail/user/megacz/allmail", false);
+ .getFileBasedMailbox("/afs/megacz.com/mail/user/megacz/allmail.sqlite", false);
- public static final int GRAYLIST_MINWAIT = 1000 * 60 * 60; // one hour
- public static final int GRAYLIST_MAXWAIT = 1000 * 60 * 60 * 24 * 5; // five days
-
- public static final int RETRY_TIME = 1000 * 60 * 30;
+ public static final int NUM_OUTGOING_THREADS = 5;
+ public static final int GRAYLIST_MINWAIT = 1000 * 60 * 60; // one hour
+ public static final int GRAYLIST_MAXWAIT = 1000 * 60 * 60 * 24 * 5; // five days
+ public static final int MAX_MESSAGE_SIZE = Integer.parseInt(System.getProperty("org.ibex.mail.smtp.maxMessageSize", "-1"));
+ public static final int RETRY_TIME = 1000 * 60 * 30; // 30min recommended by RFC
+ public static final int GIVE_UP_TIME = 1000 * 60 * 24 * 5; // FIXME: actually use this
public static final Graylist graylist;
public static final Whitelist whitelist;
}
}
- public static final int MAX_MESSAGE_SIZE =
- Integer.parseInt(System.getProperty("org.ibex.mail.smtp.maxMessageSize", "-1"));
-
private static final Mailbox spool =
FileBasedMailbox.getFileBasedMailbox(Mailbox.STORAGE_ROOT,false).slash("spool",true).slash("smtp",true).getMailbox();
- static {
- for(int i=0; i<numOutgoingThreads; i++)
- new Outgoing().start();
- }
public static void enqueue(Message m) throws IOException {
if (!m.envelopeTo.isLocal()) Outgoing.enqueue(m);
// Outgoing Mail Thread //////////////////////////////////////////////////////////////////////////////
+ static {
+ for(int i=0; i<NUM_OUTGOING_THREADS; i++)
+ new Outgoing("#"+i).start();
+ }
+
public static class Outgoing extends Thread {
+ private static HashSet<Outgoing> threads = new HashSet<Outgoing>();
private static final HashMap deadHosts = new HashMap();
+ private static Map<String,Long> nextTry = Collections.synchronizedMap(new HashMap<String,Long>());
+
+ private Mailbox.Iterator it;
+ private final String name;
+
+ public Outgoing(String name) {
+ this.name = name;
+ synchronized(Outgoing.class) {
+ threads.add(this);
+ }
+ }
+
+ public String toString() { return name; }
+
public static void enqueue(Message m) throws IOException {
if (m == null) { Log.warn(Outgoing.class, "attempted to enqueue(null)"); return; }
String traces = m.headers.get("Received");
return accepted;
}
- private static HashSet<Outgoing> threads = new HashSet<Outgoing>();
- private static int serials = 1;
- private int serial = serials++;
- private Mailbox.Iterator it;
-
- private static Map<String,Long> nextTry = Collections.synchronizedMap(new HashMap<String,Long>());
-
- 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");
+ Log.info(SMTP.Outgoing.class, "outgoing thread "+name+" woke up; " + count + " messages to send");
try {
while(true) {
boolean good = false;
//if (e instanceof InterruptedException) throw e;
Log.error(SMTP.Outgoing.class, e);
}
- Log.info(SMTP.Outgoing.class, "outgoing thread #"+serial+" going back to sleep");
+ Log.info(SMTP.Outgoing.class, "outgoing thread #"+name+" going back to sleep");
it = null;
}
public void run() {
try {
while(true) {
- Log.setThreadAnnotation("[outgoing #"+serial+"] ");
+ Log.setThreadAnnotation("[outgoing #"+name+"] ");
wake();
Thread.sleep(1000);
synchronized(Outgoing.class) {