Daniel Perry commented on JAMES-426:
------------------------------------

I agree.  The code i provided shouldn't affect any user of james, unless they 
add the extra attribute, so therefore could be added without causing any 
problems.  I would suggest that it is put to a vote on the dev list, but as i 
am not a commiter, I dont really have a say.

Daniel.

I'm -0 on this patch.

It is true that this is backward compatible but if we introduce this way to handle the servernames we should take care to keep compatibility with this "hack" in future releases.

I don't like that "servernames" depends on VirtuserTable because they are unrelated services. I would not review/commit the current patch but I don't even put any veto on this.

I'm +0 on a modular way to handle the server names:
A modular way would be to create a DomainListProvider service and to add this dependency to the james server. Then create a default implementation that read the localservernames from the xml, and another one that read this list running a query on the virtusertable.

This feature is not in my priorities and I don't have time to write it, but I can help by reviewing and committing the code (if someone code it).

Stefano

Attached is a patch that makes james use the virtualusertable database for 
identifying local hosts.  It also allows for continued use of servernames 
block.  It is backwards compatible, and unless an extra attribute is added in 
the configuration, it wont affect james's behaviour.
It is configured by adding the domainTable attribute to the servernames 
element, eg:
<servernames autodetect="true" autodetectIP="true" 
domainTable="db://maildb/VirtualUserTable">
Please note that the patch i will attach against svn trunk, but has been copied 
over directly from 2.2.0, and i havnt had a chance to check if it compiles or 
works with svn trunk.
Daniel.


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

Reply via email to