Norman Maurer (JIRA) wrote: > > Norman Maurer reassigned JAMES-472: > ----------------------------------- > > Assign To: Norman Maurer > > > Add Greylist support > > -------------------- > > > > Key: JAMES-472 > > URL: http://issues.apache.org/jira/browse/JAMES-472 > > Project: James > > Type: New Feature > > > Reporter: Norman Maurer > > Assignee: Norman Maurer > > Fix For: 2.4.0 > > > > > Im working on a patch to add Greylisting support to james. > It 90 % done. Im using a dir in which a new file is created > with the a name that looks like "ipaddress-sender-rcpt". Then > check for the modifytime to see if the "triplet" is valid and > the email now get accepted. I did it that way cause there is > no way to use sql in smtpserver at this time. > > Any problems which can caused by that ? > > If not i will post the patch soon after add junit tests
Why does this have to be in SMTPServer? These extra hits on the file system will create a bottleneck for those not interested in greylist support. Isn't this is Mailet level functionality? If there are things you can only pick up during SMTP processing we should add them as MailAttributes for detection in the Mailet chain. Generally speaking its good to decouple the mail logic from the datastore to provide deployment choices. Creating a new file in a FileStore does not scale as well as creating a new row in a database table, also searching is less performant. By making the store configurable at deployment time we can accomodate both low and high volume users. Cheers -- Steve --------------------------------------------------------------------- To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]
