james-server-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Stefano Bagnara (JIRA)" <server-...@james.apache.org>
Subject [jira] Commented: (JAMES-451) Check for valid domain in HELO
Date Sun, 26 Mar 2006 16:18:33 GMT
    [ http://issues.apache.org/jira/browse/JAMES-451?page=comments#action_12371901 ] 

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

>From a quick review I see the code has problems.
E.g: Configuration.getChild NEVER returns null (read the javadocs), so the check to see if
the child exists or not should be made differently.
I have no time to fix it and check the correctness of the code now.

As you can see this is assigned, don't ask for updates to issues: we have hundreds of issues,
if everyone asked for updates I would pass my time reading comments ;-) .

Don't take me wrong but it's not as easy as to apply and commit: I write tests for new features,
we should add documentation and comments in the config.xml to let the user know he can use
2 different helo handlers. If we add this code as is, it will serve no one but you and me
and it will be one more class in our repository to be mantained.

Please, if you have time, check it works with no "checkValidHelo" child, add unittests to
the SMTPServerTest to test both behaviours and add comment in the default config.xml. We can
keep the HelpCmdHandler name and simply add the feature to that class (disabled by default).


> 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: 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: server-dev-unsubscribe@james.apache.org
For additional commands, e-mail: server-dev-help@james.apache.org


Mime
View raw message