Need to unwrap exception before checking for security exception #816

Merged
merged 1 commit into from Jun 22, 2013

Conversation

Projects
None yet
2 participants
@ratnikov
Contributor

ratnikov commented Jun 20, 2013

As is the instanceof SecurityException is useless since reflection invocation will likely be wrapped in InvocationTargetException

@headius

This comment has been minimized.

Show comment
Hide comment
@headius

headius Jun 22, 2013

Member

Looks good to me. Merging.

Member

headius commented Jun 22, 2013

Looks good to me. Merging.

headius added a commit that referenced this pull request Jun 22, 2013

Merge pull request #816 from ratnikov/fix-logger-instantiation
Need to unwrap exception before checking for security exception

@headius headius merged commit 5031dca into jruby:master Jun 22, 2013

1 check failed

default The Travis CI build failed
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment