[
https://issues.apache.org/jira/browse/RANGER-2944?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17173587#comment-17173587
]
Jiayi Liu commented on RANGER-2944:
-----------------------------------
[~madhan]
I found another bug related to this issue. I originally wanted to update it on
this JIRA, but the patch has been committed, and I opened a new JIRA. Can you
take a look? https://issues.apache.org/jira/browse/RANGER-2945
> Presto plugin lookupResource throws NPE if not set service password
> -------------------------------------------------------------------
>
> Key: RANGER-2944
> URL: https://issues.apache.org/jira/browse/RANGER-2944
> Project: Ranger
> Issue Type: Bug
> Components: plugins
> Reporter: Jiayi Liu
> Priority: Major
> Fix For: 2.1.0, 3.0.0
>
> Attachments: 0001-RANGER-2944.patch
>
>
> In the configuration of Ranger Presto plugin service, password is not a
> required content. Only when SSL is enabled in Presto, we need to fill in the
> password. When Presto does not enable SSL, if you fill in the password when
> configuring the service, the connection will fail and an exception will be
> thrown.
> {code:java}
> java.sql.SQLException: Authentication using username/password requires SSL to
> be enabled
> {code}
> However, currently in the code, we always put
> {{prop.getProperty(HadoopConfigHolder.RANGER_LOGIN_PASSWORD)}} to
> prestoProperties, if the password is not set,
> {{prop.getProperty(HadoopConfigHolder.RANGER_LOGIN_PASSWORD)}} will be null.
> If we put a null value to {{Properties}}, this will throw NPE. We need to
> check whether we have set the password.
--
This message was sent by Atlassian Jira
(v8.3.4#803005)
