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

        Markus Wiederkehr created issue -
        Hide
        Markus Wiederkehr added a comment -

        Proposed patch..

        Show
        Markus Wiederkehr added a comment - Proposed patch..
        Markus Wiederkehr made changes -
        Field Original Value New Value
        Attachment james-889.patch [ 12405924 ]
        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.
        Markus Wiederkehr made changes -
        Attachment james-889.patch [ 12406034 ]
        Markus Wiederkehr made changes -
        Attachment james-889.patch [ 12405924 ]
        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
        Markus Wiederkehr made changes -
        Status Open [ 1 ] Resolved [ 5 ]
        Fix Version/s Trunk [ 12312135 ]
        Resolution Fixed [ 1 ]
        Norman Maurer made changes -
        Fix Version/s 3.0-M1 [ 12314294 ]
        Fix Version/s Trunk [ 12312135 ]
        Fix Version/s 2.3.2 [ 12312493 ]
        Mark Thomas made changes -
        Workflow jira [ 12461234 ] Default workflow, editable Closed status [ 12566523 ]
        Mark Thomas made changes -
        Workflow Default workflow, editable Closed status [ 12566523 ] jira [ 12582071 ]
        Transition Time In Source Status Execution Times Last Executer Last Execution Date
        Open Open Resolved Resolved
        2d 1h 52m 1 Markus Wiederkehr 22/Apr/09 14:42

          People

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

            Dates

            • Created:
              Updated:
              Resolved:

              Development