Opened 14 years ago

Closed 14 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 Changed 14 years ago by Nicklas Nordborg

Status: newassigned

comment:2 Changed 14 years ago by Nicklas Nordborg

Resolution: fixed
Status: assignedclosed

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

Note: See TracTickets for help on using tickets.