Github user zjffdu commented on the issue:
https://github.com/apache/zeppelin/pull/3003
LGTM, one suggestion is that I think it is time for us to do refactoring
for the authentication component, we should put it in one component and provide
interface to be used by other components. Instead of putting logic in zeppelin
server like this PR. ---
