Skip to content

[HOTFIX][ZEPPELIN-3526] fix when no shiro.ini exists#3011

Closed
prabhjyotsingh wants to merge 1 commit intoapache:masterfrom
prabhjyotsingh:hotfix/ZEPPELIN-3526
Closed

[HOTFIX][ZEPPELIN-3526] fix when no shiro.ini exists#3011
prabhjyotsingh wants to merge 1 commit intoapache:masterfrom
prabhjyotsingh:hotfix/ZEPPELIN-3526

Conversation

@prabhjyotsingh
Copy link
Copy Markdown
Contributor

What is this PR for?

This is a side effect of ZEPPELIN-3526, occurs when there's no shiro.ini in the classpath.

What type of PR is it?

[Hot Fix]

How should this be tested?

  • CI should be green

Questions:

  • Does the licenses files need update? n/a
  • Is there breaking changes for older versions? n/a
  • Does this needs documentation? n/a

Change-Id: I0e2bc5794182c75c5912d7478ef54a3f56fc1be1
@zjffdu
Copy link
Copy Markdown
Contributor

zjffdu commented Jun 8, 2018

LGTM, wait for CI pass

@asfgit asfgit closed this in 9fbcacf Jun 8, 2018
asfgit pushed a commit that referenced this pull request Jun 8, 2018
This is a side effect of ZEPPELIN-3526, occurs when there's no shiro.ini in the classpath.

[Hot Fix]

* CI should be green

* Does the licenses files need update? n/a
* Is there breaking changes for older versions? n/a
* Does this needs documentation? n/a

Author: Prabhjyot Singh <[email protected]>

Closes #3011 from prabhjyotsingh/hotfix/ZEPPELIN-3526 and squashes the following commits:

ac6565c [Prabhjyot Singh] fix ZEPPELIN-3526 when no shiro.ini exists

Change-Id: I5016e293eeec17e44be29dbf7f2668ec542a8dfa
(cherry picked from commit 9fbcacf)
Signed-off-by: Prabhjyot Singh <[email protected]>

# Conflicts:
#	zeppelin-server/src/main/java/org/apache/zeppelin/server/ZeppelinServer.java
prabhjyotsingh added a commit to prabhjyotsingh/zeppelin that referenced this pull request Jul 4, 2018
This is a side effect of ZEPPELIN-3526, occurs when there's no shiro.ini in the classpath.

[Hot Fix]

* CI should be green

* Does the licenses files need update? n/a
* Is there breaking changes for older versions? n/a
* Does this needs documentation? n/a

Author: Prabhjyot Singh <[email protected]>

Closes apache#3011 from prabhjyotsingh/hotfix/ZEPPELIN-3526 and squashes the following commits:

ac6565c [Prabhjyot Singh] fix ZEPPELIN-3526 when no shiro.ini exists

Change-Id: I5016e293eeec17e44be29dbf7f2668ec542a8dfa
(cherry picked from commit 9fbcacf)
Signed-off-by: Prabhjyot Singh <[email protected]>
mckartha pushed a commit to syntechdev/zeppelin that referenced this pull request Aug 9, 2018
This is a side effect of ZEPPELIN-3526, occurs when there's no shiro.ini in the classpath.

[Hot Fix]

* CI should be green

* Does the licenses files need update? n/a
* Is there breaking changes for older versions? n/a
* Does this needs documentation? n/a

Author: Prabhjyot Singh <[email protected]>

Closes apache#3011 from prabhjyotsingh/hotfix/ZEPPELIN-3526 and squashes the following commits:

ac6565c [Prabhjyot Singh] fix ZEPPELIN-3526 when no shiro.ini exists

Change-Id: I5016e293eeec17e44be29dbf7f2668ec542a8dfa
mckartha pushed a commit to syntechdev/zeppelin that referenced this pull request Aug 9, 2018
This is a side effect of ZEPPELIN-3526, occurs when there's no shiro.ini in the classpath.

[Hot Fix]

* CI should be green

* Does the licenses files need update? n/a
* Is there breaking changes for older versions? n/a
* Does this needs documentation? n/a

Author: Prabhjyot Singh <[email protected]>

Closes apache#3011 from prabhjyotsingh/hotfix/ZEPPELIN-3526 and squashes the following commits:

ac6565c [Prabhjyot Singh] fix ZEPPELIN-3526 when no shiro.ini exists

Change-Id: I5016e293eeec17e44be29dbf7f2668ec542a8dfa
(cherry picked from commit 9fbcacf)
Signed-off-by: Prabhjyot Singh <[email protected]>

# Conflicts:
#	zeppelin-server/src/main/java/org/apache/zeppelin/server/ZeppelinServer.java
@prabhjyotsingh prabhjyotsingh deleted the hotfix/ZEPPELIN-3526 branch May 2, 2022 16:24
Sign up for free to 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