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

        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
        Mark Thomas made changes -
        Workflow Default workflow, editable Closed status [ 12566523 ] jira [ 12582071 ]
        Mark Thomas made changes -
        Workflow jira [ 12461234 ] Default workflow, editable Closed status [ 12566523 ]
        Norman Maurer made changes -
        Fix Version/s 3.0-M1 [ 12314294 ]
        Fix Version/s Trunk [ 12312135 ]
        Fix Version/s 2.3.2 [ 12312493 ]
        Markus Wiederkehr made changes -
        Status Open [ 1 ] Resolved [ 5 ]
        Fix Version/s Trunk [ 12312135 ]
        Resolution Fixed [ 1 ]
        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
        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.
        Markus Wiederkehr made changes -
        Attachment james-889.patch [ 12405924 ]
        Markus Wiederkehr made changes -
        Attachment james-889.patch [ 12406034 ]
        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
        Robert Burrell Donkin added a comment -

        +1 on this patch

        • robert
        Show
        Robert Burrell Donkin added a comment - +1 on this patch robert
        Markus Wiederkehr made changes -
        Field Original Value New Value
        Attachment james-889.patch [ 12405924 ]
        Hide
        Markus Wiederkehr added a comment -

        Proposed patch..

        Show
        Markus Wiederkehr added a comment - Proposed patch..
        Markus Wiederkehr created issue -

          People

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

            Dates

            • Created:
              Updated:
              Resolved:

              Development