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.
---
