You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@activemq.apache.org by "ASF GitHub Bot (Jira)" <ji...@apache.org> on 2021/01/08 20:35:00 UTC

[jira] [Work logged] (ARTEMIS-3058) Improper AddressSettings merge

     [ https://issues.apache.org/jira/browse/ARTEMIS-3058?focusedWorklogId=533212&page=com.atlassian.jira.plugin.system.issuetabpanels:worklog-tabpanel#worklog-533212 ]

ASF GitHub Bot logged work on ARTEMIS-3058:
-------------------------------------------

                Author: ASF GitHub Bot
            Created on: 08/Jan/21 20:34
            Start Date: 08/Jan/21 20:34
    Worklog Time Spent: 10m 
      Work Description: jbertram opened a new pull request #3401:
URL: https://github.com/apache/activemq-artemis/pull/3401


   The merge method in AddressSettings should *not* use any getters. It
   should reference the relevant variables directly. Using any getters will
   return default values in the underlying value is null. This can cause
   problems for hierarchical settings.
   
   Also fixed a few potential NPEs exposed by the test-case.


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


Issue Time Tracking
-------------------

            Worklog Id:     (was: 533212)
    Remaining Estimate: 0h
            Time Spent: 10m

> Improper AddressSettings merge
> ------------------------------
>
>                 Key: ARTEMIS-3058
>                 URL: https://issues.apache.org/jira/browse/ARTEMIS-3058
>             Project: ActiveMQ Artemis
>          Issue Type: Bug
>            Reporter: Justin Bertram
>            Assignee: Justin Bertram
>            Priority: Major
>          Time Spent: 10m
>  Remaining Estimate: 0h
>




--
This message was sent by Atlassian Jira
(v8.3.4#803005)