[ https://issues.apache.org/jira/browse/AMQ-4428?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
Christoffer Sawicki updated AMQ-4428: ------------------------------------- Description: {{setProperty}} currently ends with {{ … catch (Throwable ignore) \{ return false; \} }}. This is really evil since it can make ActiveMQ ignore invalid configuration completely silently. I think that errors should at least be logged. (Is there any case when this isn't wanted?) (was: `setProperty` currently ends with `… catch (Throwable ignore) { return false; }`. This is really evil since it can make ActiveMQ ignore invalid configuration completely silently. I think that errors should at least be logged. (Is there any case when this isn't wanted?)) > Don't swallow errors in IntrospectionSupport.setProperty > -------------------------------------------------------- > > Key: AMQ-4428 > URL: https://issues.apache.org/jira/browse/AMQ-4428 > Project: ActiveMQ > Issue Type: Bug > Reporter: Christoffer Sawicki > Priority: Minor > Attachments: 0001-Log-errors-in-IntrospectionSupport.setProperty.patch > > > {{setProperty}} currently ends with {{ … catch (Throwable ignore) \{ return > false; \} }}. This is really evil since it can make ActiveMQ ignore invalid > configuration completely silently. I think that errors should at least be > logged. (Is there any case when this isn't wanted?) -- This message is automatically generated by JIRA. If you think it was sent incorrectly, please contact your JIRA administrators For more information on JIRA, see: http://www.atlassian.com/software/jira