Skip to content

ZEPPELIN-3332 Zeppelin login fails with NPE if ldapRealm.authorizationEnabled is not set true - #2911

Closed
felixalbani wants to merge 3 commits into
apache:masterfrom
felixalbani:ZEPPELIN-3332
Closed

ZEPPELIN-3332 Zeppelin login fails with NPE if ldapRealm.authorizationEnabled is not set true#2911
felixalbani wants to merge 3 commits into
apache:masterfrom
felixalbani:ZEPPELIN-3332

Conversation

@felixalbani

Copy link
Copy Markdown
Contributor

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

bag_of_tricks added 3 commits April 5, 2018 12:21
Simply fixes NPE by checking for null
This reverts commit dd7d37d.
Simply fixes NPE by checking for null
@felixalbanifelixalbani changed the title Zeppelin 3332ZEPPELIN-3332 Zeppelin login fails with NPE if ldapRealm.authorizationEnabled is not set trueApr 5, 2018
@felixalbani

Copy link
Copy Markdown
ContributorAuthor

I created this new PR since previous PR #2867 was not created from master. Sorry for any inconveniences.

@felixalbani

Copy link
Copy Markdown
ContributorAuthor

@Leemoonsoo

@Leemoonsoo

Copy link
Copy Markdown
Member

Thanks @felixalbani. merging it.

asfgit pushed a commit that referenced this pull request Apr 5, 2018
…nEnabled is not set true
### 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
Author: bag_of_tricks <falbani@hortonworks.com>
Closes#2911 from felixalbani/ZEPPELIN-3332 and squashes the following commits:
c6ccf41 [bag_of_tricks] ZEPPELIN-3332622333e [bag_of_tricks] Revert "ZEPPELIN-3332"
dd7d37d [bag_of_tricks] ZEPPELIN-3332
(cherry picked from commit 685eb92)
Signed-off-by: Lee moon soo <moon@apache.org>
@asfgitasfgit closed this in 685eb92Apr 5, 2018
jwagun pushed a commit to jwagun/zeppelin that referenced this pull request Apr 23, 2018
…nEnabled is not set true
### 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
Author: bag_of_tricks <falbani@hortonworks.com>
Closesapache#2911 from felixalbani/ZEPPELIN-3332 and squashes the following commits:
c6ccf41 [bag_of_tricks] ZEPPELIN-3332
622333e [bag_of_tricks] Revert "ZEPPELIN-3332"
dd7d37d [bag_of_tricks] ZEPPELIN-3332
prabhjyotsingh pushed a commit to prabhjyotsingh/zeppelin that referenced this pull request Jul 4, 2018
…nEnabled is not set true
Simply fixes NPE by checking for null
Bug Fix
https://issues.apache.org/jira/browse/ZEPPELIN-3332
Does the licenses files need update? No
Is there breaking changes for older versions? No
Does this needs documentation? No
Author: bag_of_tricks <falbani@hortonworks.com>
Closesapache#2911 from felixalbani/ZEPPELIN-3332 and squashes the following commits:
c6ccf41 [bag_of_tricks] ZEPPELIN-3332
622333e [bag_of_tricks] Revert "ZEPPELIN-3332"
dd7d37d [bag_of_tricks] ZEPPELIN-3332
(cherry picked from commit 685eb92)
Signed-off-by: Lee moon soo <moon@apache.org>
(cherry picked from commit d3de515)
Change-Id: I169443b32613376ce3a7e40d4a956bb0d194d04e
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

@felixalbani@Leemoonsoo