You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@zeppelin.apache.org by jongyoul <gi...@git.apache.org> on 2018/12/03 05:22:45 UTC
[GitHub] zeppelin pull request #3246: ZEPPELIN-3892 Make SecurityUtils as a service
GitHub user jongyoul opened a pull request:
https://github.com/apache/zeppelin/pull/3246
ZEPPELIN-3892 Make SecurityUtils as a service
### What is this PR for?
Making SecurityService more configurable. Currently, SecurityUtils consists of static methods, and which means it's not extendable and pluggable. It would be better if users could implement new security features easily.
### What type of PR is it?
[Refactoring]
### Todos
* [x] - Introduce `SecurityService` as an interface to support security features
* [x] - Move `SecurityUtils` to `ShiroSecurityService` as an implementation of `SecurityService`
### What is the Jira issue?
* https://issues.apache.org/jira/browse/ZEPPELIN-3892
### How should this be tested?
* This is a refactoring. It should pass all of the current tests
### 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/jongyoul/zeppelin ZEPPELIN-3892
Alternatively you can review and apply these changes as the patch at:
https://github.com/apache/zeppelin/pull/3246.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 #3246
----
commit 422d169fa3df35ea0b1ee72084a88c7651f7943a
Author: Jongyoul Lee <jo...@...>
Date: 2018-12-03T01:53:53Z
Introduce an interface of SecurityService to handle issues of securities
Change SecurityUtils to ShiroSecurityService
Introduce NoSecurityService to divide to handle without shiro configuration
commit 723c388c680e121ca1f0dedaeb68e21a522aaa31
Author: Jongyoul Lee <jo...@...>
Date: 2018-12-03T01:56:34Z
Add licenses
commit fa2dd487856fc3bc04478d756103df5d9badac4a
Author: Jongyoul Lee <jo...@...>
Date: 2018-12-03T02:06:12Z
Move validation logic for shiro to ShiroSecurityService
commit 932b1f6783be3a84598bcfce8b35c8b889840c2e
Author: Jongyoul Lee <jo...@...>
Date: 2018-12-03T02:25:36Z
Add license
----
---
[GitHub] zeppelin issue #3246: ZEPPELIN-3892 Make SecurityUtils as a service
Posted by jongyoul <gi...@git.apache.org>.
Github user jongyoul commented on the issue:
https://github.com/apache/zeppelin/pull/3246
Ready to review. Failed tests might be related to #3243
---
[GitHub] zeppelin pull request #3246: ZEPPELIN-3892 Make SecurityUtils as a service
Posted by jongyoul <gi...@git.apache.org>.
GitHub user jongyoul reopened a pull request:
https://github.com/apache/zeppelin/pull/3246
ZEPPELIN-3892 Make SecurityUtils as a service
### What is this PR for?
Making SecurityService more configurable. Currently, SecurityUtils consists of static methods, and which means it's not extendable and pluggable. It would be better if users could implement new security features easily.
### What type of PR is it?
[Refactoring]
### Todos
* [x] - Introduce `SecurityService` as an interface to support security features
* [x] - Move `SecurityUtils` to `ShiroSecurityService` as an implementation of `SecurityService`
### What is the Jira issue?
* https://issues.apache.org/jira/browse/ZEPPELIN-3892
### How should this be tested?
* This is a refactoring. It should pass all of the current tests
### 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/jongyoul/zeppelin ZEPPELIN-3892
Alternatively you can review and apply these changes as the patch at:
https://github.com/apache/zeppelin/pull/3246.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 #3246
----
commit c5022b2cc1559eabca1bd647fd0ff1a004644198
Author: Jongyoul Lee <jo...@...>
Date: 2018-12-03T01:53:53Z
Introduce an interface of SecurityService to handle issues of securities
Change SecurityUtils to ShiroSecurityService
Introduce NoSecurityService to divide to handle without shiro configuration
commit 37a00d08080944ef7e9abbb09fa9fe9b7bdb7ebc
Author: Jongyoul Lee <jo...@...>
Date: 2018-12-03T01:56:34Z
Add licenses
commit f253cf4986ae0a902f0664e0a3c9db9242a00888
Author: Jongyoul Lee <jo...@...>
Date: 2018-12-03T02:06:12Z
Move validation logic for shiro to ShiroSecurityService
commit ca72b2040b85f35c61e8d250bc61cb80f33d3589
Author: Jongyoul Lee <jo...@...>
Date: 2018-12-03T02:25:36Z
Add license
commit 74d5d653378afbdc007c65be379920855d49701b
Author: Jongyoul Lee <jo...@...>
Date: 2018-12-03T13:30:25Z
Change ShiroSecurityService to SecurityService
commit dab7e64a8ead41d38e9f2312814f6286dac7bddb
Author: Jongyoul Lee <jo...@...>
Date: 2018-12-03T13:36:47Z
Extract two more methods to the interface
commit 70cfb04d56c0f9ec682f70e48fe9cf1f5687a617
Author: Jongyoul Lee <jo...@...>
Date: 2018-12-03T14:36:34Z
Fix injection information
----
---
[GitHub] zeppelin pull request #3246: ZEPPELIN-3892 Make SecurityUtils as a service
Posted by jongyoul <gi...@git.apache.org>.
Github user jongyoul closed the pull request at:
https://github.com/apache/zeppelin/pull/3246
---
[GitHub] zeppelin pull request #3246: ZEPPELIN-3892 Make SecurityUtils as a service
Posted by asfgit <gi...@git.apache.org>.
Github user asfgit closed the pull request at:
https://github.com/apache/zeppelin/pull/3246
---
[GitHub] zeppelin issue #3246: ZEPPELIN-3892 Make SecurityUtils as a service
Posted by jongyoul <gi...@git.apache.org>.
Github user jongyoul commented on the issue:
https://github.com/apache/zeppelin/pull/3246
https://travis-ci.org/jongyoul/zeppelin/builds/462832712
---