Im not a developer to james, but i answer cause i submit it ;-) I choose to add it to main cause i thought some guys found it a bit confusing to read about HELO/EHLO in the MailCmdHandler. But i have no probs to put it to the Handler ( that how i did for the other features i add).
I also see no negative thinks on move the options to the handlers. About the postmaster handling: I don't see why we should accept emails without HELO/EHLO if HeloEhloEnforcement is activated. Cause HELO/EHLO must be send (RFC)! I don't think that we should accept it for the other cases (sender domain mx check, resolveable HELO/EHLO) cause if a admin consider to use this feature he don't want to not use them on postmaster acounts (maybe im to lazy, but i think thats what admins are ;-)!). But thats just me thinking. bye Norman Ps: Keep up the nice work guys Am Samstag, den 15.04.2006, 16:41 +0200 schrieb Stefano Bagnara: > Norman Maurer wrote: > > I will add this improvment to james as fast as i can.. But i think we > > should add this config value do james core and not the smtphandler chain > > cause otherwise it whould be in the mail handler (thats a bit > > confusing).. What you think ? > > I applied the patch as it was (configuration option in the main handler). > > I think it would be better to move the option to the MailCmdHandler, but > I would like to know what other developers/users think about this. > > We have more similar commands that could be moved to specific handlers. > > lengthReset (DATA) > verifyIdentity (RCPT) > ehloHeloEnforcement (MAIL) > > While the following require iteraction between handlers. > > authorizedAddresses + authRequired (EHLO, MAIL, RCPT) > MaxMessageSize (EHLO, MAIL, DATA, OnMessage) > > Another thing we should consider is the postmaster recipient handling: > RFC says we should always accept mails destinated to postmaster. How > should we handle EHLO/HELO enforcements, MAIL FROM domain validation, > EHLO/HELO domain validation and that RFC rule? > > Stefano > > > --------------------------------------------------------------------- > To unsubscribe, e-mail: [EMAIL PROTECTED] > For additional commands, e-mail: [EMAIL PROTECTED] > > !EXCUBATOR:1,444106a337021224119642!
signature.asc
Description: Dies ist ein digital signierter Nachrichtenteil
