[#JAMES-490] Javamail 1.4 regression: SocketFetcher changed to require SocketFactory implementation

JAMES Server

Javamail 1.4 regression: SocketFetcher changed to require SocketFactory implementation

Created: 11/May/06 10:54 PM Updated: 21/Nov/07 08:31 AM

Return to search

Component/s:

Remote Delivery

Affects Version/s:

2.3.0

Fix Version/s:

2.3.0

Time Tracking:

Not Specified

Resolution Date:

13/May/06 09:00 PM

Description

« Hide

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.

All

Comments

Work Log

Change History

Subversion Commits

Sort Order:

[ Permlink | « Hide ]

Stefano Bagnara added a comment - 11/May/06 10:56 PM

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.

[ Permlink | « Hide ]

Stefano Bagnara added a comment - 13/May/06 09:00 PM

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

[ Permlink | « Hide ]

Danny Angus added a comment - 21/Nov/07 08:31 AM

Closing issue fixed in released version.