[ 
https://issues.apache.org/jira/browse/QPID-5738?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15171213#comment-15171213
 ] 

ASF subversion and git services commented on QPID-5738:
-------------------------------------------------------

Commit 1732808 from [~k-wall] in branch 'java/trunk'
[ https://svn.apache.org/r1732808 ]

QPID-5738: [Java Broker] Model - remove setAttribute and replace with 
setAttributes

> [Java Broker] AbstractConfigureObject#setAttribute() does not validate 
> attributes
> ---------------------------------------------------------------------------------
>
>                 Key: QPID-5738
>                 URL: https://issues.apache.org/jira/browse/QPID-5738
>             Project: Qpid
>          Issue Type: Bug
>          Components: Java Broker
>    Affects Versions: 0.29
>            Reporter: Keith Wall
>            Assignee: Rob Godfrey
>         Attachments: 
> 0001-QPID-5738-Java-Broker-Model-remove-setAttribute-and-.patch, 
> setAttributeRefactor.patch
>
>
> Code path ACO#setAttribute omits to invoke the attribute validation 
> facilities of ACO.   This means that there are circumstances (mainly via the 
> JMX interface) where an unvalidated attribute value can be set on a 
> configured object.
> We decided that in the short term, ACO#setAttribute() should delegate to its 
> pluralised version (which does validate attributes).    We tried, I found a 
> couple of tests fell foul of the validations and needed remediated.  The 
> problem also highlighted AbstractQueue's exclusive attribute, whose 
> validations appears to need need some work to conform to the new model.
> WIP patch attached.
>  



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to