[jira] [Commented] (RANGER-2944) Presto plugin lookupResource throws NPE if not set service password

["Jiayi Liu (Jira)" <ji...@apache.org>]

    [ 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)