Hi Norman, 1) I think that the correct name is RoamingUsers (note the G after roamin).
I don't know if "RoamingUsers" is better or worst than "PopBeforeSmtp": what do ther thinks?
I like RoamingUsers as a name but I don't know what would be the most intuitive name to our users.
2) Before 3.0 I would like to find a better solution than hardcoding the check in PassCmdHandler (maybe support the new fastfail in pop3, and add support for filters after command execution for this). Maybe we should also think at the possibility to use a single handler as smtp and pop3 filter at the same time... This would be a much more complex issue, but much better than using a static map.
3) We start having a little mess with authorized users, relaying allowed, authorized networks and how handlers manage this. Let's keep in mind this and maybe we'll find a clean solution to this.
Stefano PS: I think this is a good feature to James! [EMAIL PROTECTED] wrote:
Author: norman Date: Sun Jul 23 04:10:51 2006 New Revision: 424709 URL: http://svn.apache.org/viewvc?rev=424709&view=rev Log: Add support for RoaminUsers (pop before smtp). See JAMES-569 Added: james/server/trunk/src/java/org/apache/james/smtpserver/core/RoaminUsersHandler.java james/server/trunk/src/java/org/apache/james/util/RoaminUsersHelper.java Modified: james/server/trunk/src/conf/james-config.xml james/server/trunk/src/java/org/apache/james/pop3server/PassCmdHandler.java james/server/trunk/src/java/org/apache/james/smtpserver/SMTPHandler.java james/server/trunk/src/java/org/apache/james/smtpserver/SMTPServer.java james/server/trunk/src/java/org/apache/james/smtpserver/SMTPSession.java james/server/trunk/src/test/org/apache/james/pop3server/POP3ServerTest.java james/server/trunk/src/test/org/apache/james/smtpserver/DNSRBLHandlerTest.java james/server/trunk/src/test/org/apache/james/smtpserver/SPFHandlerTest.java james/server/trunk/src/test/org/apache/james/smtpserver/SetMimeHeaderHandlerTest.java
--------------------------------------------------------------------- To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]
