You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@ranger.apache.org by pr...@apache.org on 2022/02/07 08:35:45 UTC

[ranger] branch master updated: RANGER-3542: Fix invalid HTTPS check

This is an automated email from the ASF dual-hosted git repository.

pradeep pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/ranger.git


The following commit(s) were added to refs/heads/master by this push:
     new 2607038  RANGER-3542: Fix invalid HTTPS check
2607038 is described below

commit 26070383c6300da91926ed77e128d35c9808056c
Author: pradeep <pr...@apache.org>
AuthorDate: Thu Feb 3 19:08:02 2022 +0530

    RANGER-3542: Fix invalid HTTPS check
---
 .../main/java/org/apache/ranger/plugin/util/RangerRESTClient.java   | 6 +++++-
 1 file changed, 5 insertions(+), 1 deletion(-)

diff --git a/agents-common/src/main/java/org/apache/ranger/plugin/util/RangerRESTClient.java b/agents-common/src/main/java/org/apache/ranger/plugin/util/RangerRESTClient.java
index be9b826..49e1281 100644
--- a/agents-common/src/main/java/org/apache/ranger/plugin/util/RangerRESTClient.java
+++ b/agents-common/src/main/java/org/apache/ranger/plugin/util/RangerRESTClient.java
@@ -250,7 +250,7 @@ public class RangerRESTClient {
 			LOG.error("RangerRESTClient.init(): failed to create GsonBuilder object", excp);
 		}
 
-		mIsSSL = StringUtils.containsIgnoreCase(mUrl, "https");
+		mIsSSL = isSsl(mUrl);
 
 		if (mIsSSL) {
 
@@ -281,6 +281,10 @@ public class RangerRESTClient {
 		}
 	}
 
+	private boolean isSsl(String url) {
+		return !StringUtils.isEmpty(url) && url.toLowerCase().startsWith("https");
+	}
+
 	private KeyManager[] getKeyManagers() {
 		KeyManager[] kmList = null;