Opened 16 years ago

Closed 16 years ago

#1042 closed enhancement (fixed)

Improve error handling in the extensions system

Reported by: Nicklas Nordborg Owned by: Nicklas Nordborg
Priority: major Milestone: BASE 2.8
Component: web Version:
Keywords: Cc:

Description

The extensions system contains code like the example below in several places. It may seem ok, but one common exception is a NoClassDefFoundError (happens when a JAR file is missing), which is not caught by the RuntimeException. This messes up the extension system and a single extension that fails may affect other extensions. I think that in most cases it would be more appropriate to catch Throwable instead. 

try
{
  // code...
}
catch (RuntimeException ex)
{
  log.error(....);
}

Change History (2)

comment:1 by Nicklas Nordborg, 16 years ago

Status: new → assigned

comment:2 by Nicklas Nordborg, 16 years ago

Resolution: → fixed
Status: assigned → closed

(In [4322]) Fixes #1042: Improve error handling in the extensions system

Note: See TracTickets for help on using tickets.