Issue Details (XML | Word | Printable)

Key: JAMES-522
Type: Bug Bug
Status: Closed Closed
Resolution: Fixed
Priority: Trivial Trivial
Assignee: Norman Maurer
Reporter: Noel J. Bergman
Votes: 0
Watchers: 0
Operations

If you were logged in you would be able to see more operations.
JAMES Server

Having the ClamAVScan mailet configured, but clamd unavailable when JAMES starts, keeps JAMES from starting.

Created: 04/Jun/06 09:30 AM   Updated: 21/Nov/07 08:31 AM
Return to search
Component/s: Matchers/Mailets (bundled)
Affects Version/s: 2.3.0
Fix Version/s: 2.3.0

Time Tracking:
Not Specified

Resolution Date: 06/Jun/06 02:36 PM


 Description  « Hide
If JAMES attempts to start, ClamAVScan is configured, and clamd is not running, ClamAVScan.init() will throw an exception, terminating JAMES entirely. This can be seen in ${james}/logs/phoenix.log where the maxPings error will appear, and the spoolmanager component will be shown as failing.

I'm marking this as minor since there is a work around, biut actually, I consider it a more major error.

 All   Comments   Work Log   Change History   Subversion Commits      Sort Order: Ascending order - Click to sort in descending order
Noel J. Bergman added a comment - 04/Jun/06 09:35 AM
In the mailet config, adding <maxPings>0</maxPings> will disable the check, and the error, so this is arguably a "trivial" error. Still, it presents a good example of where we can improve retry with queuing. And we really should provide a better error message to help admins, since the default is to fail.

Vincenzo Gianferrari Pini added a comment - 05/Jun/06 07:43 PM
I would keep things as now, because this was the intended behaviour. If <maxPings>n</maxPings>, with n > 0, the desired behaviour is to delay the startup of James until clamd is up and running, *without allowing any message to flow without an anti-virus check*. Such situation could arise with a server startup, where clamd could take longer than james to start. If after some time clamd has not yet started, James must fail and the administrator shall intervene to fix clamd.
If I don't want such "safe" behaviour, I can set <maxPings>0</maxPings> as you did.
BTW, I'm changing config.xml to have a commented invocation of ClamAVScan (plus some other enhancements).

Danny Angus added a comment - 21/Nov/07 08:31 AM
Closing issue fixed in released version.