Eric, Yes, thanks for the correction, I mean empty MAIL FROM (in other words "MAIL FROM <>".
You are already handling the situation where a mail client/relay is not passing a MAIL FROM at all properly (by responding with "503 5.5.0 Need MAIL before RCPT"). I'll open the ticket and be more precise. Thanks, Aaron > -----Original Message----- > From: Eric Charles [mailto:[email protected]] > Sent: Tuesday, June 26, 2012 1:34 PM > To: James Users List > Subject: Re: Question about empty MAIL FROM > > Hi Aaron, > Yes, please open the ticket on JAMES. > I suppose you mean 'empty' MAIL FROM, and not 'null'. > > In other words, in case of MAIL FROM command not issued, do we have to > throw and exception, and if yes, which one? > > (sorry, no time atm to dig into the RFC) > > Thx, Eric > > On 06/26/2012 05:57 PM, Aaron Freeman wrote: > > I'll be happy to open a ticket (though I haven't done one before so > > don't hate me if I screw it up a bit) -- which component do you think > > is more appropriate, JAMES or PROTOCOL? I'll essentially just include > > this thread, including Stefano's responses -- I can't add much beyond > > his statement that > > RFC1123 5.2.9 says it's a must and that setting a null MAIL FROM is a > > common technique for preventing bounce loops. > > > > Thanks, > > > > Aaron > > > > > >> -----Original Message----- > >> From: Eric Charles [mailto:[email protected]] > >> Sent: Tuesday, June 26, 2012 4:04 AM > >> To: James Users List > >> Subject: Re: Question about empty MAIL FROM > >> > >> Hi Aaron, > >> > >> Sorry to return you the question, but what does SMTP RFC > >> (http://www.ietf.org/rfc/rfc2821.txt) tells about empty mails? (is it > >> allowed?) > >> > >> In any case, it would be better to a more specific and throw an > >> exception with a correct readable message. > >> > >> Based on your finding, it will be good to open a JIRA on > >> https://issues.apache.org/jira/browse/JAMES or > >> https://issues.apache.org/jira/browse/PROTOCOLS (it will depend > >> how/where we will fix it). > >> > >> Thx, Eric > >> > >> On 06/25/2012 05:39 PM, Aaron Freeman wrote: > >>> On James Server 3.0 beta 4, we are seeing lots of these errors in > >>> the > > logs: > >>> > >>> > >>> ERROR 10:31:12,847 | james.smtpserver | Id='234288323' User='' > >>> Invalid email address while processing DATA. > >>> javax.mail.internet.AddressException: Out of data at position 1 in '' > >>> in string ``'' at position 1 > >>> at > >>> org.apache.mailet.MailAddress.<init>(MailAddress.java:159) > >>> > >>> So I tried to reproduce it and while I can't be 100% sure this is > >>> what is causing it, it does at least reproduce it: > >>> > >>> [root@alpha log]# telnet 10.123.123.123 25 Trying 10.123.123.123... > >>> Connected to alpha.nospam.com (10.123.123.123). > >>> Escape character is '^]'. > >>> 220 alpha.nospam.com JAMES SMTP Server Server (JAMES SMTP Server > ) > >>> ready EHLO testcomputer.com 250-alpha.nospam.com Hello > >>> testcomputer.com [10.123.123.123]) 250-AUTH LOGIN PLAIN 250- > >> AUTH=LOGIN > >>> PLAIN 250-PIPELINING 250-ENHANCEDSTATUSCODES > >>> 250 8BITMIME > >>> MAIL FROM:<> > >>> 250 2.1.0 Sender<> OK > >>> RCPT TO:<[email protected]> > >>> 250 2.1.5 Recipient<[email protected]> OK DATA > >>> 354 Ok Send data ending with<CRLF>.<CRLF> > >>> Subject: Test > >>> > >>> Test > >>> . > >>> 451 4.0.0 Error processing message: Out of data at position 1 in '' > >>> quit > >>> 451 Unable to process request > >>> Connection closed by foreign host. > >>> > >>> Is there a setting that will allow us to process emails with an > >>> empty > > MAIL > >>> FROM, or is this just a bug that needs to be submitted somewhere? > >>> > >>> Thoughts? > >>> > >>> Aaron > >>> > >>> > >>> > >>> -------------------------------------------------------------------- > >>> - To unsubscribe, e-mail: [email protected] > >>> For additional commands, e-mail: [email protected] > >>> > >> > >> -- > >> eric | http://about.echarles.net | @echarles > >> > >> --------------------------------------------------------------------- > >> To unsubscribe, e-mail: [email protected] > >> For additional commands, e-mail: [email protected] > > > > > > --------------------------------------------------------------------- > > To unsubscribe, e-mail: [email protected] > > For additional commands, e-mail: [email protected] > > > > -- > eric | http://about.echarles.net | @echarles > > --------------------------------------------------------------------- > To unsubscribe, e-mail: [email protected] > For additional commands, e-mail: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
