[RANGER-2944] Presto plugin lookupResource throws NPE if not set service password

• Type: Bug
• Status: Resolved
• Priority: Major
• Resolution: Fixed
• Fix Version/s: 2.1.0, 3.0.0
• Component/s: plugins
• Assignee: Jiayi Liu
• Reporter: Jiayi Liu
• Created: 08/Aug/20 04:24
• Updated: 09/Aug/20 17:49
• Resolved: 08/Aug/20 05:35

Description

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.

java.sql.SQLException: Authentication using username/password requires SSL to be enabled

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.

Issue Links

• links to: Review Request