Uploaded image for project: 'James Server'
  1. James Server
  2. JAMES-490

Javamail 1.4 regression: SocketFetcher changed to require SocketFactory implementation

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Blocker
    • Resolution: Fixed
    • Affects Version/s: 2.3.0
    • Fix Version/s: 2.3.0
    • Component/s: Remote Delivery
    • Labels:
      None

      Description

      The SocketFetcher used by Javamail 1.4 does no more use reflection to call the socket creation method on the SocketFactory provided. It instead cast the created object to SocketFactory.
      Out SocketFactory does not implement extends the SocketFactory class so it does not connect and fail to send mails.

      The socketFactory is only used when a bindAddress is specified, but this should be fixed asap.

        Activity

        Hide
        bago Stefano Bagnara added a comment -

        One more note: SocketFetcher in 1.3.2 and 1.4 supports the usage of the "mail.smtp.localaddress" properties to specify a bind address. Maybe we don't need to use our local factory now.

        Show
        bago Stefano Bagnara added a comment - One more note: SocketFetcher in 1.3.2 and 1.4 supports the usage of the "mail.smtp.localaddress" properties to specify a bind address. Maybe we don't need to use our local factory now.
        Hide
        bago Stefano Bagnara added a comment -

        This is resolved, althought I would probably remove the factory in favor of "mail.smtp.localaddress" session property.

        Show
        bago Stefano Bagnara added a comment - This is resolved, althought I would probably remove the factory in favor of "mail.smtp.localaddress" session property.
        Hide
        danny@apache.org Danny Angus added a comment -

        Closing issue fixed in released version.

        Show
        danny@apache.org Danny Angus added a comment - Closing issue fixed in released version.

          People

          • Assignee:
            bago Stefano Bagnara
            Reporter:
            bago Stefano Bagnara
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development