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

Posted to dev@storm.apache.org by revans2 <gi...@git.apache.org> on 2018/09/11 15:55:39 UTC

[GitHub] storm pull request #2824: STORM-3218: Return the correct error from Authoriz...

GitHub user revans2 opened a pull request:

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

    STORM-3218: Return the correct error from AuthorizedUserFilter

    The changes to ReqContext are not needed.  I started to make a change there, and cleaned it up before realizing that it was not going to fix the problem, but I decided to leave the code cleanup in place.
    
    I also fixed a few other issues with AuthorizedUserFilter including leaking a nimbus client, and not returning after aborting the request, which was resulting in an NPE in some cases.

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

    $ git pull https://github.com/revans2/incubator-storm STORM-3218

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

    https://github.com/apache/storm/pull/2824.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 #2824
    
----
commit 38d59713486e1d74edae92e1577bf2d0b3942b22
Author: Robert (Bobby) Evans <ev...@...>
Date:   2018-09-11T14:40:17Z

    STORM-3218: Return the correct error from AuthorizedUserFilter

----


---

[GitHub] storm pull request #2824: STORM-3218: Return the correct error from Authoriz...

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

Github user asfgit closed the pull request at:

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


---