[ 
http://issues.apache.org/jira/browse/JAMES-451?page=comments#action_12372437 ] 

Stefano Bagnara commented on JAMES-451:
---------------------------------------

I committed the latest patches but now a test fail.
I had to apply your patches manually because the format was not correctly 
handled by Eclipse: can you check I committed all the code and why the test 
doesn't pass?

testEhloResolv(), line 332:

junit.framework.AssertionFailedError: expected error: ehlo could not resolved 
expected:<501> but was:<250>
        at junit.framework.Assert.fail(Assert.java:47)
        at junit.framework.Assert.failNotEquals(Assert.java:282)
        at junit.framework.Assert.assertEquals(Assert.java:64)
        at junit.framework.Assert.assertEquals(Assert.java:201)
        at 
org.apache.james.smtpserver.SMTPServerTest.testEhloResolv(SMTPServerTest.java:332)
        at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
        at sun.reflect.NativeMethodAccessorImpl.invoke(Unknown Source)
        at sun.reflect.DelegatingMethodAccessorImpl.invoke(Unknown Source)
        at java.lang.reflect.Method.invoke(Unknown Source)
        at junit.framework.TestCase.runTest(TestCase.java:154)
        at junit.framework.TestCase.runBare(TestCase.java:127)
        at junit.framework.TestResult$1.protect(TestResult.java:106)
        at junit.framework.TestResult.runProtected(TestResult.java:124)
        at junit.framework.TestResult.run(TestResult.java:109)
        at junit.framework.TestCase.run(TestCase.java:118)
        at junit.framework.TestSuite.runTest(TestSuite.java:208)
        at junit.framework.TestSuite.run(TestSuite.java:203)
        at 
org.eclipse.jdt.internal.junit.runner.RemoteTestRunner.runTests(RemoteTestRunner.java:478)
        at 
org.eclipse.jdt.internal.junit.runner.RemoteTestRunner.run(RemoteTestRunner.java:344)
        at 
org.eclipse.jdt.internal.junit.runner.RemoteTestRunner.main(RemoteTestRunner.java:196)



> Check for valid domain in HELO
> ------------------------------
>
>          Key: JAMES-451
>          URL: http://issues.apache.org/jira/browse/JAMES-451
>      Project: James
>         Type: Wish
>   Components: SMTPServer
>     Reporter: Norman Maurer
>     Assignee: Stefano Bagnara
>  Attachments: ehlo-junit.patch, ehlo.patch, helo-v2-junit.patch, 
> helo-v2.patch, helo.patch
>
> IT would be nice to let 
> /james-dev/src/java/org/apache/james/smtpserver/HeloCmdHandler.java to accept 
> an init parameter for specify that the helo value must be an resolvable 
> domainname. In RFC HELO names should always a valid domainname.

-- 
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators:
   http://issues.apache.org/jira/secure/Administrators.jspa
-
For more information on JIRA, see:
   http://www.atlassian.com/software/jira


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

Reply via email to