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 <jongyoul@...>
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 <jongyoul@...>
Date:   2018-12-03T01:56:34Z

    Add licenses

commit f253cf4986ae0a902f0664e0a3c9db9242a00888
Author: Jongyoul Lee <jongyoul@...>
Date:   2018-12-03T02:06:12Z

    Move validation logic for shiro to ShiroSecurityService

commit ca72b2040b85f35c61e8d250bc61cb80f33d3589
Author: Jongyoul Lee <jongyoul@...>
Date:   2018-12-03T02:25:36Z

    Add license

commit 74d5d653378afbdc007c65be379920855d49701b
Author: Jongyoul Lee <jongyoul@...>
Date:   2018-12-03T13:30:25Z

    Change ShiroSecurityService to SecurityService

commit dab7e64a8ead41d38e9f2312814f6286dac7bddb
Author: Jongyoul Lee <jongyoul@...>
Date:   2018-12-03T13:36:47Z

    Extract two more methods to the interface

commit 70cfb04d56c0f9ec682f70e48fe9cf1f5687a617
Author: Jongyoul Lee <jongyoul@...>
Date:   2018-12-03T14:36:34Z

    Fix injection information

----


---

Reply via email to