[ 
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

Reply via email to