MrTopicConnection.java-fixed string comparion 71/75271/1
authorDriptaroop Das <driptaroop.das@in.ibm.com>
Fri, 4 Jan 2019 07:54:23 +0000 (13:24 +0530)
committerDriptaroop Das <driptaroop.das@in.ibm.com>
Fri, 4 Jan 2019 07:55:17 +0000 (13:25 +0530)
MrTopicConnection.java - Move the "https" string literal on the left side of this string comparison.

Issue-ID: DMAAP-931
Change-Id: Ia8217bd68c63e07a45cb064bf77bdf91934ffdff
Signed-off-by: Driptaroop Das <driptaroop.das@in.ibm.com>
src/main/java/org/onap/dmaap/dbcapi/client/MrTopicConnection.java

index 492037c..a92dbc7 100644 (file)
@@ -3,6 +3,8 @@
  * org.onap.dmaap
  * ================================================================================
  * Copyright (C) 2017 AT&T Intellectual Property. All rights reserved.
+ *
+ * Modifications Copyright (C) 2019 IBM.
  * ================================================================================
  * Licensed under the Apache License, Version 2.0 (the "License");
  * you may not use this file except in compliance with the License.
@@ -63,7 +65,7 @@ public class MrTopicConnection extends BaseLoggingClass  {
 
                topicURL = cluster.getTopicProtocol() + "://" + fqdn + ":" + cluster.getTopicPort() + "/events/" + topic ;
 
-               if ( cluster.getTopicProtocol().equals( "https")) {
+               if ( "https".equals(cluster.getTopicProtocol())) {
                        return makeSecureConnection( topicURL );
                }
                return makeConnection( topicURL );