|
Hi all,
Was looking in Bugzilla to see if there was
anything I could take a stab at. Attached is a diff for bug 2006. This should
get MailAddress to accept localpart@[xxx.xxx.xxx.xxx]
addresses.
diff.txt patches MailAddress to make it properly
parse out the dotNum host part. diff2.txt is a patch to RemoteDelivery to have
it strip the [] chars from the host before doing getMailServers, so the lookup
works.
If the fixes are acceptable, I'm not sure what
the protocol is, but the bug could be closed in bugzilla.
Have done some local testing, and will keep hacking
at it to see if there are any other parts choking on the [xxx.xxx.xxx.xxx] host
parts.
Matt Pangaro
|
Index: RemoteDelivery.java
===================================================================
RCS file:
/home/cvspublic/jakarta-james/src/java/org/apache/james/transport/mailets/RemoteDelivery.java,v
retrieving revision 1.7
diff -r1.7 RemoteDelivery.java
134a135,139
> //Strip [] chars from dotNum addresses
> if (host.startsWith("[") && host.endsWith("]")) {
> host = host.substring(1, host.length() - 1);
> }
>Index: MailAddress.java =================================================================== RCS file: /home/cvspublic/jakarta-james/src/java/org/apache/mailet/MailAddress.java,v retrieving revision 1.2 diff -r1.2 MailAddress.java 327a328,332 > //we were passed the string with pos pointing the the [ char. > // take the first char ([), put it in the result buffer and increment >pos > resultSB.append(address.charAt(pos)); > pos++; >
--------------------------------------------------------------------- To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]
