[GitHub] zeppelin issue #1757: [ZEPPELIN-1074] Enhance SSL Support

2016-12-20 Thread lresende
Github user lresende commented on the issue: https://github.com/apache/zeppelin/pull/1757 No worries @AhyoungRyu I will make the suggested changes and add ZEPPELIN-1811 to this pr as well. --- If your project is set up for it, you can reply to this email and have your reply appear on

[GitHub] zeppelin issue #1757: [ZEPPELIN-1074] Enhance SSL Support

2016-12-18 Thread AhyoungRyu
Github user AhyoungRyu commented on the issue: https://github.com/apache/zeppelin/pull/1757 Sorry for my late blocking, but actually I even didn't know that there is `README.md` under `conf/`. Maybe many ppl are same with me I guess. Seems there is already some duplicated information

[GitHub] zeppelin issue #1757: [ZEPPELIN-1074] Enhance SSL Support

2016-12-18 Thread felixcheung
Github user felixcheung commented on the issue: https://github.com/apache/zeppelin/pull/1757 merging if no more comment --- 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 an

[GitHub] zeppelin issue #1757: [ZEPPELIN-1074] Enhance SSL Support

2016-12-18 Thread felixcheung
Github user felixcheung commented on the issue: https://github.com/apache/zeppelin/pull/1757 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 feature enabled and wishes so, or if t

[GitHub] zeppelin issue #1757: [ZEPPELIN-1074] Enhance SSL Support

2016-12-15 Thread Leemoonsoo
Github user Leemoonsoo commented on the issue: https://github.com/apache/zeppelin/pull/1757 Thanks @lresende for enhancement! Looks good to me. --- 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

[GitHub] zeppelin issue #1757: [ZEPPELIN-1074] Enhance SSL Support

2016-12-15 Thread lresende
Github user lresende commented on the issue: https://github.com/apache/zeppelin/pull/1757 @Leemoonsoo Could you please take a quick look at this. Thank you. --- 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

[GitHub] zeppelin issue #1757: [ZEPPELIN-1074] Enhance SSL Support

2016-12-15 Thread lresende
Github user lresende commented on the issue: https://github.com/apache/zeppelin/pull/1757 @vinayshukla I have extended the README.md to detail how to obfuscate the passwords using the Jetty password tool --- If your project is set up for it, you can reply to this email and have your

[GitHub] zeppelin issue #1757: [ZEPPELIN-1074] Enhance SSL Support

2016-12-15 Thread vinayshukla
Github user vinayshukla commented on the issue: https://github.com/apache/zeppelin/pull/1757 @lresende In case the password is obfuscated by Jetty's tool, is there any code required to de-obfuscate the password before passing it onto SSLContext? --- If your project is set up for it,