Fix security vulnerability 56/2756/1
authorRehanRaza <muhammad.rehan.raza@est.tech>
Wed, 11 Mar 2020 15:24:21 +0000 (16:24 +0100)
committerRehanRaza <muhammad.rehan.raza@est.tech>
Wed, 11 Mar 2020 15:25:07 +0000 (16:25 +0100)
Change-Id: I7e07370f8a3f5ae4f20f535f805c672c43047aa8
Issue-ID: NONRTRIC-141
Signed-off-by: RehanRaza <muhammad.rehan.raza@est.tech>
dashboard/webapp-backend/src/main/java/org/oransc/ric/portal/dashboard/util/HttpsURLConnectionUtils.java

index b86bbfe..32646de 100644 (file)
@@ -41,7 +41,8 @@ public final class HttpsURLConnectionUtils {
 
     private static final HostnameVerifier jvmHostnameVerifier = HttpsURLConnection.getDefaultHostnameVerifier();
 
-    private static final HostnameVerifier trivialHostnameVerifier = (hostname, sslSession) -> true;
+    private static final HostnameVerifier trivialHostnameVerifier =
+        (hostname, sslSession) -> hostname.equalsIgnoreCase(sslSession.getPeerHost());
 
     private static final TrustManager[] UNQUESTIONING_TRUST_MANAGER = new TrustManager[] {new X509TrustManager() {
         @SuppressWarnings("squid:S1168") // Must return null to get wanted behaviour.