Author: norman
Date: Sat Jan 27 14:05:11 2007
New Revision: 500636

URL: http://svn.apache.org/viewvc?view=rev&rev=500636
Log:
Not accept recipient after unresolvable domain. See JAMES-771

Modified:
    
james/server/trunk/src/java/org/apache/james/smtpserver/core/filter/fastfail/ValidSenderDomainHandler.java
    james/server/trunk/src/test/org/apache/james/smtpserver/SMTPServerTest.java

Modified: 
james/server/trunk/src/java/org/apache/james/smtpserver/core/filter/fastfail/ValidSenderDomainHandler.java
URL: 
http://svn.apache.org/viewvc/james/server/trunk/src/java/org/apache/james/smtpserver/core/filter/fastfail/ValidSenderDomainHandler.java?view=diff&rev=500636&r1=500635&r2=500636
==============================================================================
--- 
james/server/trunk/src/java/org/apache/james/smtpserver/core/filter/fastfail/ValidSenderDomainHandler.java
 (original)
+++ 
james/server/trunk/src/java/org/apache/james/smtpserver/core/filter/fastfail/ValidSenderDomainHandler.java
 Sat Jan 27 14:05:11 2007
@@ -115,6 +115,7 @@
             }
         
             if (records == null || records.size() == 0) {
+               session.getState().remove(SMTPSession.SENDER);
                 return true;
             }
         }

Modified: 
james/server/trunk/src/test/org/apache/james/smtpserver/SMTPServerTest.java
URL: 
http://svn.apache.org/viewvc/james/server/trunk/src/test/org/apache/james/smtpserver/SMTPServerTest.java?view=diff&rev=500636&r1=500635&r2=500636
==============================================================================
--- james/server/trunk/src/test/org/apache/james/smtpserver/SMTPServerTest.java 
(original)
+++ james/server/trunk/src/test/org/apache/james/smtpserver/SMTPServerTest.java 
Sat Jan 27 14:05:11 2007
@@ -490,13 +490,12 @@
         smtpProtocol1.helo(InetAddress.getLocalHost().toString());
 
         String sender1 = "[EMAIL PROTECTED]";
-        String sender2 = "[EMAIL PROTECTED]";
         
         smtpProtocol1.setSender(sender1);
         assertEquals("expected 501 error", 501, smtpProtocol1.getReplyCode());
-    
-        smtpProtocol1.setSender(sender2);
-
+        
+        smtpProtocol1.addRecipient("[EMAIL PROTECTED]");
+        assertEquals("Recipient not accepted cause no valid sender", 503, 
smtpProtocol1.getReplyCode());
         smtpProtocol1.quit();
         
     }



---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]

Reply via email to