[GitHub] zeppelin issue #2911: ZEPPELIN-3332 Zeppelin login fails with NPE if ldapRea...

[felixalbani]

Github user felixalbani commented on the issue:

https://github.com/apache/zeppelin/pull/2911
  
@Leemoonsoo 


---

[GitHub] zeppelin issue #2911: ZEPPELIN-3332 Zeppelin login fails with NPE if ldapRea...

[felixalbani]

Github user felixalbani commented on the issue:

https://github.com/apache/zeppelin/pull/2911
  
I created this new PR since previous PR 
https://github.com/apache/zeppelin/pull/2867 was not created from master. Sorry 
for any inconveniences.



---

[GitHub] zeppelin pull request #2911: Zeppelin 3332

[felixalbani]

GitHub user felixalbani opened a pull request:

https://github.com/apache/zeppelin/pull/2911

Zeppelin 3332

### What is this PR for?
Simply fixes NPE by checking for null

### What type of PR is it?
Bug Fix

### What is the Jira issue?
https://issues.apache.org/jira/browse/ZEPPELIN-3332

### Questions:
Does the licenses files need update? No
Is there breaking changes for older versions? No
Does this needs documentation? No

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

$ git pull https://github.com/felixalbani/zeppelin ZEPPELIN-3332

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

https://github.com/apache/zeppelin/pull/2911.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 #2911


commit dd7d37dac991fc04ec6287fe05235e027e20e1f2
Author: bag_of_tricks <falbani@...>
Date:   2018-04-05T16:21:23Z

ZEPPELIN-3332

Simply fixes NPE by checking for null

commit 622333eb925870b5b4a2ce22da063b9857dd3da1
Author: bag_of_tricks <falbani@...>
Date:   2018-04-05T16:24:01Z

Revert "ZEPPELIN-3332"

This reverts commit dd7d37dac991fc04ec6287fe05235e027e20e1f2.

commit c6ccf4189ad50c09749b2a06d5fb61830075ab84
Author: bag_of_tricks <falbani@...>
Date:   2018-04-05T16:27:18Z

ZEPPELIN-3332

Simply fixes NPE by checking for null




---

[GitHub] zeppelin pull request #2867: ZEPPELIN-3332 Zeppelin login fails with NPE if ...

[felixalbani]

Github user felixalbani closed the pull request at:

https://github.com/apache/zeppelin/pull/2867


---

[GitHub] zeppelin pull request #2867: ZEPPELIN-3332

[felixalbani]

GitHub user felixalbani opened a pull request:

https://github.com/apache/zeppelin/pull/2867

ZEPPELIN-3332

Zeppelin login fails with NPE if ldapRealm.authorizationEnabled is not set 
true

Issue was introduced by pull https://github.com/apache/zeppelin/pull/2498

### What is this PR for?
A few sentences describing the overall goals of the pull request's commits.
First time? Check out the contributing guide - 
https://zeppelin.apache.org/contribution/contributions.html

### What type of PR is it?
Bug Fix

### Todos
* [ ] - Task

### What is the Jira issue?
https://issues.apache.org/jira/browse/ZEPPELIN-3332

### How should this be tested?
* First time? Setup Travis CI as described on 
https://zeppelin.apache.org/contribution/contributions.html#continuous-integration
* Strongly recommended: add automated unit tests for any new or changed 
behavior
* Outline any manual steps to test the PR here.

### Screenshots (if appropriate)

### Questions:
* Does the licenses files need update? No
* Is there breaking changes for older versions? No
* Does this needs documentation? No


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

$ git pull https://github.com/felixalbani/zeppelin ZEPPELIN-3332

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

https://github.com/apache/zeppelin/pull/2867.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 #2867


commit ceb7b6f837cfe196fd1a9b51abbffe0c31ee7e71
Author: bag_of_tricks <falbani@...>
Date:   2018-03-14T14:44:25Z

ZEPPELIN-3332

Zeppelin login fails with NPE if ldapRealm.authorizationEnabled is not set 
true

Issue was introduced by pull https://github.com/apache/zeppelin/pull/2498




---