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

ASF GitHub Bot commented on STORM-565:
--------------------------------------

GitHub user harshach opened a pull request:

    https://github.com/apache/storm/pull/323

    STORM-565. Setting topology.groups to nil could break SimpleACLAuthorizer

    

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

    $ git pull https://github.com/harshach/incubator-storm STORM-565

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

    https://github.com/apache/storm/pull/323.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 #323
    
----
commit 5c0a56105a9a984ae8e693343a6d89b730fd2ff9
Author: Sriharsha Chintalapani <[email protected]>
Date:   2014-11-20T16:36:17Z

    STORM-565. Setting topology.groups to nil could break SimpleACLAuthorizer.

----


> Setting topology.groups to nil could break SimpleACLAuthorizer
> --------------------------------------------------------------
>
>                 Key: STORM-565
>                 URL: https://issues.apache.org/jira/browse/STORM-565
>             Project: Apache Storm
>          Issue Type: Bug
>    Affects Versions: 0.9.3-rc2
>            Reporter: Derek Dagit
>            Assignee: Sriharsha Chintalapani
>            Priority: Minor
>              Labels: security
>
> https://github.com/apache/storm/blob/f8bce2225f2cfc550dcc67dff469f87062686506/storm-core/src/jvm/backtype/storm/security/auth/authorizer/SimpleACLAuthorizer.java#L114
> If a user would happen to include this config, but set its value to nil, then 
> we could see an NPE here.



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

Reply via email to