Stefano Bagnara schrieb:
> Norman,
>
> this is v2.3, a branch in RC state!!
> bugfixes should be applied to trunk and *then* we should discuss
> wether to backport to v2.3, IMHO.
Ups... You 100 % right... shame on me. For sure i should commit to trunk
first.  I will revert it in branch and start apply the changes in trunk
again, after we discuss it!

>
> Then you used tabs instead of spaces, but this is another issue ;-)
Strange.. I have to check my eclipse


>
> About the issue and the patch, looking at the code it seems that it
> should have already worked, so probably if we have a bug it is related
> to DNS and not to fetchmail. In the current fetchmail code there are
> almost 30 lines of code to manage the -1 scenario, so it seems a bit
> weird to simply add a couple of lines to avoid their execution.
> Maybe Steve can help understanding what's going on...
>
> Have you debugged the problem? Where exactly the unkownhost happens?

Well, I think its unneeded to parse the the header and try to resolve
the ip/domain if index -1 is used. Its a waste of  ressource. The
problem appeared here with exchange server message which not had an ip
address in the Received header. For me it should not fail to fetch this
emails if index -1 is used because the comment tells me that it will use
127.0.0.1 and localhost everytime. So why parse it ?

See https://issues.apache.org/jira/browse/JAMES-787

bye
Norman




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

Reply via email to