GitHub user prasadwagle reopened a pull request: https://github.com/apache/incubator-zeppelin/pull/860
[ZEPPELIN-828] Use individual user credentials for data source authentication ### What is this PR for? The purpose of this pull request is to allow users to use their own credentials to authenticate with data sources. It contains the following: - web UI and rest API to let users input their credentials - changes to AuthenticationInfo to add data source credentials - changes to Paragraph to set data source credentials in AuthenticationInfo - changes to HiveInterpreter to use the user credentials and simplification to use a new connection for every query. We made this change since we found an issue with closed vertica connections. Since we are using HiveInterpreter for analytics, the cost of creating a new connection is small compared to query execution. We don't have to merge this change into master. ### What type of PR is it? Feature ### Todos * [x] - Save user credentials in a file * [x] - Do not persist AuthenticationInfo in paragraph since it contains sensitive information * [ ] - Add tests * [ ] - Add documentation ### What is the Jira issue? [ZEPPELIN-828] (https://issues.apache.org/jira/browse/ZEPPELIN-828) ### How should this be tested? 1. In conf/shiro.ini, use /** = authc 1. Login to Zeppelin 2. Enter credentials for a data source 3. Check if the interpreter is using the correct credentials ### Screenshots (if appropriate) https://gfycat.com/LiquidGentleKatydid ### Questions: * Does this needs documentation? Yes. You can merge this pull request into a Git repository by running: $ git pull https://github.com/prasadwagle/incubator-zeppelin ZEPPELIN-828 Alternatively you can review and apply these changes as the patch at: https://github.com/apache/incubator-zeppelin/pull/860.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 #860 ---- commit 50368e9223e4b5cb56adff259492ccf27ed8725c Author: Prasad Wagle <pwa...@twitter.com> Date: 2016-04-27T20:10:00Z Use individual user credentials for data source authentication commit ecd6adeff3adaf265f0991284c328ef0d9106f5b Author: Prasad Wagle <pwa...@twitter.com> Date: 2016-05-09T23:11:05Z Merge branch 'master' into ZEPPELIN-828 commit cdb870c68401afa9b90cf8947b552bb12c119998 Author: Prasad Wagle <pwa...@twitter.com> Date: 2016-05-10T18:31:08Z Pass all user credentials to the interpreter commit 5442fc1b7446114494f18afda4febdc6976d212d Author: Prasad Wagle <pwa...@twitter.com> Date: 2016-05-10T20:43:33Z Persist user info instead of authentication info in paragraph commit 6f2f2eead2b69e1931802497febb8d4100c9e8a7 Author: Prasad Wagle <pwa...@twitter.com> Date: 2016-05-10T22:02:05Z Persist credentials depending on conf setting commit 8d2abf402ac5663a5186856f0e62b1ca9d72653f Author: Prasad Wagle <pwa...@twitter.com> Date: 2016-05-11T00:15:07Z Fix checkstyle error, Update Data Source Authorization section in docs commit dde01576b8a2515f2344ad6650337506d150c0a2 Author: Prasad Wagle <pwa...@twitter.com> Date: 2016-05-11T00:28:36Z Remove vertica dependency from hive/pom.xml commit 087bfdb7fc9dd66e601ea12207deeef8d1db8e37 Author: Prasad Wagle <pwa...@twitter.com> Date: 2016-05-11T21:24:51Z Revert changes to HiveInterpreter, Update tests commit 9427663f9bb2987c2cbc72e7f989c36d5427d162 Author: Prasad Wagle <pwa...@twitter.com> Date: 2016-05-11T23:48:30Z Added conf/credentials.json to .gitignore commit a49ef7e885dac75d12174f3e2fad91133f057c23 Author: Prasad Wagle <pwa...@twitter.com> Date: 2016-05-21T23:00:08Z Use a more general term 'entity' instead of 'datasource' commit 6b265360cd8da5bc5ded92f3de194cad297ac412 Author: Prasad Wagle <pwa...@twitter.com> Date: 2016-05-21T23:17:57Z Merge with upstream/master commit 94469b06978130d8c5580826116c87651e26d247 Author: Prasad Wagle <pwa...@twitter.com> Date: 2016-05-24T23:01:19Z Use LOG.error(msg, e) commit 8add0285b1ed69fd08189b889fb585b812af613b Author: Prasad Wagle <pwa...@twitter.com> Date: 2016-05-26T20:50:03Z Merge branch 'master' into ZEPPELIN-828 commit 9a9824bf50570bbe76b8fbfc8db8549e0d3adbf0 Author: Prasad Wagle <pwa...@twitter.com> Date: 2016-05-26T21:04:30Z Merge branch 'master' into ZEPPELIN-828 ---- --- 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 feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---