AnnotatedClassActionResolver Creates An Exception, Logs The Exception, Then Throws The Exception...

Description

I think that the following log.error(...) statement should be removed:

1 2 3 4 5 6 7 8 9 10 // from: net.sourceforge.stripes.controller.AnnotatedClassActionResolver#getActionBean // ... } catch (Exception e) { StripesServletException sse = new StripesServletException( "Could not create instance of ActionBean type [" + beanClass.getName() + "].", e); log.error(sse); throw sse; } // ...

The logging of the newly created StripesServletException is excessive. The reason I bring it up is because I use Guice injection everywhere...and when a ProvisionException is thrown I can end up with thousands of lines of logged stacktrace that I can only control by turning off AnnotatedClassActionResolver logging (via log4j.xml or logback.xml).

Any chance we can get rid of this? What purpose can that log message serve?

Environment

None

Status

Assignee

BenB

Reporter

Dan Nelson

Labels

None

Tester

None

Components

Fix versions

Affects versions

Release 1.5.7

Priority

Major
Configure