[jira] [Commented] (JAMES-2097) Using full recipients for following MX iteration when partial fails on delivering

2019-09-05 Thread Tellier Benoit (Jira) 


[ 
https://issues.apache.org/jira/browse/JAMES-2097?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16923869#comment-16923869
 ] 

Tellier Benoit commented on JAMES-2097:
---

https://github.com/linagora/james-project/pull/2658 (integration test) is merged

> Using full recipients for following MX iteration when partial fails on 
> delivering
> -
>
> Key: JAMES-2097
> URL: https://issues.apache.org/jira/browse/JAMES-2097
> Project: James Server
>  Issue Type: Bug
>  Components: Remote Delivery
>Reporter: TzeKai Lee
>Priority: Major
>
> Untested, just looks weird to me when browsing code.
> In the {{SendFailedExceptionMail}} handling of {{Delivrer.doDeliver}}, there 
> is no update of recipients to valid unsent addresses like it did in the 
> exception handling of {{Delivrer.deliver}}.
> It seems to me that if dns resolved more than one MX and for some reason the 
> first MX only accept partial recipients and the other MX accept full 
> recipients, it will result duplicated mail for the partially accepted 
> recipients



--
This message was sent by Atlassian Jira
(v8.3.2#803003)

-
To unsubscribe, e-mail: server-dev-unsubscr...@james.apache.org
For additional commands, e-mail: server-dev-h...@james.apache.org

[jira] [Commented] (JAMES-2097) Using full recipients for following MX iteration when partial fails on delivering

2019-09-04 Thread Tellier Benoit (Jira) 


[ 
https://issues.apache.org/jira/browse/JAMES-2097?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16922200#comment-16922200
 ] 

Tellier Benoit commented on JAMES-2097:
---

https://github.com/linagora/james-project/pull/2658 added an integration test 
for this bug report

https://github.com/linagora/james-project/pull/2662 proposed a fix for this bug

> Using full recipients for following MX iteration when partial fails on 
> delivering
> -
>
> Key: JAMES-2097
> URL: https://issues.apache.org/jira/browse/JAMES-2097
> Project: James Server
>  Issue Type: Bug
>  Components: Remote Delivery
>Reporter: TzeKai Lee
>Priority: Major
>
> Untested, just looks weird to me when browsing code.
> In the {{SendFailedExceptionMail}} handling of {{Delivrer.doDeliver}}, there 
> is no update of recipients to valid unsent addresses like it did in the 
> exception handling of {{Delivrer.deliver}}.
> It seems to me that if dns resolved more than one MX and for some reason the 
> first MX only accept partial recipients and the other MX accept full 
> recipients, it will result duplicated mail for the partially accepted 
> recipients



--
This message was sent by Atlassian Jira
(v8.3.2#803003)

-
To unsubscribe, e-mail: server-dev-unsubscr...@james.apache.org
For additional commands, e-mail: server-dev-h...@james.apache.org

[jira] [Commented] (JAMES-2097) Using full recipients for following MX iteration when partial fails on delivering

2019-08-20 Thread Tellier Benoit (Jira) 


[ 
https://issues.apache.org/jira/browse/JAMES-2097?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16911985#comment-16911985
 ] 

Tellier Benoit commented on JAMES-2097:
---

JAMES-2853 is an attempt toprovide a tool to write such tests. We plan to soon 
test this issue.

> Using full recipients for following MX iteration when partial fails on 
> delivering
> -
>
> Key: JAMES-2097
> URL: https://issues.apache.org/jira/browse/JAMES-2097
> Project: James Server
>  Issue Type: Bug
>  Components: Remote Delivery
>Reporter: TzeKai Lee
>Priority: Major
>
> Untested, just looks weird to me when browsing code.
> In the {{SendFailedExceptionMail}} handling of {{Delivrer.doDeliver}}, there 
> is no update of recipients to valid unsent addresses like it did in the 
> exception handling of {{Delivrer.deliver}}.
> It seems to me that if dns resolved more than one MX and for some reason the 
> first MX only accept partial recipients and the other MX accept full 
> recipients, it will result duplicated mail for the partially accepted 
> recipients



--
This message was sent by Atlassian Jira
(v8.3.2#803003)

-
To unsubscribe, e-mail: server-dev-unsubscr...@james.apache.org
For additional commands, e-mail: server-dev-h...@james.apache.org

[jira] [Commented] (JAMES-2097) Using full recipients for following MX iteration when partial fails on delivering

2017-07-23 Thread Tellier Benoit (JIRA) 

[ 
https://issues.apache.org/jira/browse/JAMES-2097?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16097876#comment-16097876
 ] 

Tellier Benoit commented on JAMES-2097:
---

Hi.

Thanks for your returns. It looks like we are badly handling return codes of 
the receiving server, thus fall into the issue you describe.

Using javax for sending the message, partially sent messages generate a 
SendFailedException. We are just not removing the recipients that have 
successfully already been sent from the mail. (See 
handleSendFailExceptionOnMxIteration in MailDelivrer)

> Using full recipients for following MX iteration when partial fails on 
> delivering
> -
>
> Key: JAMES-2097
> URL: https://issues.apache.org/jira/browse/JAMES-2097
> Project: James Server
>  Issue Type: Bug
>  Components: Remote Delivery
>Reporter: TzeKai Lee
>
> Untested, just looks weird to me when browsing code.
> In the {{SendFailedExceptionMail}} handling of {{Delivrer.doDeliver}}, there 
> is no update of recipients to valid unsent addresses like it did in the 
> exception handling of {{Delivrer.deliver}}.
> It seems to me that if dns resolved more than one MX and for some reason the 
> first MX only accept partial recipients and the other MX accept full 
> recipients, it will result duplicated mail for the partially accepted 
> recipients



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)

-
To unsubscribe, e-mail: server-dev-unsubscr...@james.apache.org
For additional commands, e-mail: server-dev-h...@james.apache.org