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 <evans@...>
Date: 2018-09-11T14:40:17Z
STORM-3218: Return the correct error from AuthorizedUserFilter
----
---
