James Server
  1. James Server
  2. JAMES-899

RemoteDelivery sends a message again and again if transport.close() throws a MessagingException

    Details

    • Type: Bug Bug
    • Status: Resolved
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 2.3.1
    • Fix Version/s: 3.0-M1
    • Component/s: None
    • Labels:
      None

      Description

      Some mail servers close the TCP connection as soon as they have received and responded to the terminating dot of the DATA command. As a consequence JavaMail throws a MessagingException in SMTPTransport.close() and RemoteDelivery thinks it has to send the message again (and again).

      Even though the server's behaviour is not standard compliant (see RFC 5321 section 4.1.1.10.) the message should still be considered to be delivered successfully because the error occurred after the mail transaction has been completed (RFC 5321 section 3.3).

      1. james-889.patch
        2 kB
        Markus Wiederkehr

        Activity

        Hide
        Markus Wiederkehr added a comment -

        Proposed patch..

        Show
        Markus Wiederkehr added a comment - Proposed patch..
        Hide
        Robert Burrell Donkin added a comment -

        +1 on this patch

        • robert
        Show
        Robert Burrell Donkin added a comment - +1 on this patch robert
        Hide
        Stefano Bagnara added a comment -

        +1 but I'd add at least an e.getMessage() to the logged string.

        Show
        Stefano Bagnara added a comment - +1 but I'd add at least an e.getMessage() to the logged string.
        Hide
        Markus Wiederkehr added a comment -

        Updated patch to include the mail recipients and the detail message of the MessagingException in the log message.

        Show
        Markus Wiederkehr added a comment - Updated patch to include the mail recipients and the detail message of the MessagingException in the log message.
        Hide
        Markus Wiederkehr added a comment -

        fixed in trunk and branch v2.3

        Show
        Markus Wiederkehr added a comment - fixed in trunk and branch v2.3

          People

          • Assignee:
            Markus Wiederkehr
            Reporter:
            Markus Wiederkehr
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development