[GitHub] incubator-zeppelin pull request: [HOTFIX] Authentication doesn't w...

2016-05-03 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/incubator-zeppelin/pull/870 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the fea

[GitHub] incubator-zeppelin pull request: [HOTFIX] Authentication doesn't w...

2016-05-03 Thread Leemoonsoo
Github user Leemoonsoo commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/870#issuecomment-216632774 Thanks @prabhjyotsingh taking a look. I'm merging it into master as a hotfix --- If your project is set up for it, you can reply to this email and have yo

[GitHub] incubator-zeppelin pull request: [HOTFIX] Authentication doesn't w...

2016-05-03 Thread prabhjyotsingh
Github user prabhjyotsingh commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/870#issuecomment-216629763 Tested, works for now, and user session remains after page refresh. --- If your project is set up for it, you can reply to this email and have your re

[GitHub] incubator-zeppelin pull request: [HOTFIX] Authentication doesn't w...

2016-05-03 Thread Leemoonsoo
Github user Leemoonsoo commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/870#issuecomment-216627328 After couple of different trial, figured out apply shiro filter to `/*` as before, and use authcBasic (default browser user auth dialog) instead of authc

[GitHub] incubator-zeppelin pull request: [HOTFIX] Authentication doesn't w...

2016-05-03 Thread corneadoug
Github user corneadoug commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/870#issuecomment-216476403 Can you try to re-trigger CI? --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your proj

[GitHub] incubator-zeppelin pull request: [HOTFIX] Authentication doesn't w...

2016-05-03 Thread bzz
Github user bzz commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/870#issuecomment-216457974 [ZEPPELIN-836](https://issues.apache.org/jira/browse/ZEPPELIN-836) was created to address CI issues --- If your project is set up for it, you can reply to this

[GitHub] incubator-zeppelin pull request: [HOTFIX] Authentication doesn't w...

2016-05-02 Thread bzz
Github user bzz commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/870#issuecomment-216432006 👍 Looks great, thank you for hotfixing! It looks like it's flaky webapp build somehow. ``` olve@1.1.7, fileset@0.2.1, escodegen@1.8.0, js

[GitHub] incubator-zeppelin pull request: [HOTFIX] Authentication doesn't w...

2016-05-02 Thread jongyoul
Github user jongyoul commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/870#issuecomment-216429543 LGTM --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feat

[GitHub] incubator-zeppelin pull request: [HOTFIX] Authentication doesn't w...

2016-05-02 Thread prabhjyotsingh
Github user prabhjyotsingh commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/870#issuecomment-216427588 Update: login information does not stays after page refresh. --- If your project is set up for it, you can reply to this email and have your reply app

[GitHub] incubator-zeppelin pull request: [HOTFIX] Authentication doesn't w...

2016-05-02 Thread prabhjyotsingh
Github user prabhjyotsingh commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/870#issuecomment-216426281 Tested, LGTM. CI fails for unrelated reason. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHu

[GitHub] incubator-zeppelin pull request: [HOTFIX] Authentication doesn't w...

2016-05-02 Thread Leemoonsoo
GitHub user Leemoonsoo opened a pull request: https://github.com/apache/incubator-zeppelin/pull/870 [HOTFIX] Authentication doesn't work after upgrade to jetty9 ### What is this PR for? After the jetty9 upgrade https://github.com/apache/incubator-zeppelin/pull/831, authenticatio