You are viewing a plain text version of this content. The canonical link for it is here.

Posted to dev@activemq.apache.org by arkaev <gi...@git.apache.org> on 2018/06/06 13:24:52 UTC

[GitHub] activemq-artemis pull request #2124: Fix NPE on ConfigurationImpl.hashCode()

GitHub user arkaev opened a pull request:

    https://github.com/apache/activemq-artemis/pull/2124

    Fix NPE on ConfigurationImpl.hashCode()

    maskPassword can be null now

You can merge this pull request into a Git repository by running:

    $ git pull https://github.com/arkaev/activemq-artemis master

Alternatively you can review and apply these changes as the patch at:

    https://github.com/apache/activemq-artemis/pull/2124.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

    This closes #2124
    
----
commit 0790313452f6d8351439b49de13ef8b63a6c7888
Author: Andrey Arkaev <aa...@...>
Date:   2018-06-06T13:08:15Z

    Fix NPE on ConfigurationImpl.hashCode()
    
    maskPassword can be null now

----


---

[GitHub] activemq-artemis issue #2124: Fix NPE on ConfigurationImpl.hashCode()

Posted by clebertsuconic <gi...@git.apache.org>.

Github user clebertsuconic commented on the issue:

    https://github.com/apache/activemq-artemis/pull/2124
  
    I created a JIRA and a test.. and ammended as your commit.
    
    
    Thank you so much.



---

[GitHub] activemq-artemis pull request #2124: Fix NPE on ConfigurationImpl.hashCode()

Posted by asfgit <gi...@git.apache.org>.

Github user asfgit closed the pull request at:

    https://github.com/apache/activemq-artemis/pull/2124


---

[GitHub] activemq-artemis issue #2124: Fix NPE on ConfigurationImpl.hashCode()

Posted by arkaev <gi...@git.apache.org>.

Github user arkaev commented on the issue:

    https://github.com/apache/activemq-artemis/pull/2124
  
    it's easier to use java.util.Objects#hash


---