stale: removed stale log lines. update: version in pom. README with logo

Author: rchougule

README.md

@@ -1,3 +1,7 @@
+<p align="center">
+  <a href="https://browserstack.com"><img alt="BrowserStack Logo" src="https://d98b8t1nnulk5.cloudfront.net/production/images/layout/logo-invoice.svg"></a>
+</p>
+
 # Overview
 [BrowserStack](https://browserstack.com) gives instant access to 2000+ real mobile devices and browsers that enables developers to test their websites and mobile applications without requiring to install or maintain an internal lab of virtual machines, devices, or emulators.
 

pom.xml

@@ -10,7 +10,7 @@
   </parent>
 
   <artifactId>browserstack-integration</artifactId>
-  <version>1.1.10-SNAPSHOT</version>
+  <version>1.2.0</version>
   <packaging>hpi</packaging>
 
   <name>BrowserStack</name>
@@ -150,7 +150,7 @@
     <dependency>
       <groupId>com.browserstack</groupId>
       <artifactId>automate-client-java</artifactId>
-      <version>0.6</version>
+      <version>0.7</version>
     </dependency>
 
     <dependency>

src/main/java/com/browserstack/automate/ci/common/proxysettings/JenkinsProxySettings.java

@@ -13,7 +13,7 @@
 import java.net.URL;
 
 import static com.browserstack.automate.ci.common.logger.PluginLogger.log;
-import static com.browserstack.automate.ci.common.logger.PluginLogger.logError;
+import static com.browserstack.automate.ci.common.logger.PluginLogger.logDebug;
 
 public class JenkinsProxySettings {
 
@@ -79,9 +79,8 @@ private URL verifyAndGetProxyURL(final String proxyString, final String proxyTyp
 
             return proxyUrl;
         } catch (Exception e) {
-            // TODO: Change to logDebug
             if (logger != null)
-                logError(logger, String.format("Invalid Proxy String: %s, Proxy Type: %s. Error: %s", proxyString, proxyType, e.toString()));
+                logDebug(logger, String.format("Invalid Proxy String: %s, Proxy Type: %s. Error: %s", proxyString, proxyType, e.toString()));
             return null;
         }
     }
@@ -129,9 +128,6 @@ private void decideJenkinsProxy(final String customProxyString) {
                 this.finalProxyUsername = jarProxyUser;
                 this.finalProxyPassword = jarProxyPassword;
             }
-
-            if (this.finalProxyUsername == null)
-                System.out.println("Username is null in case of JAR proxy...");
         }
 
         // Utilises the proxyUrl set by Env Vars if Jenkins & Jar Proxy are absent
@@ -180,6 +176,7 @@ private String getSystemProxyString() {
 
     /**
      * Returns Jenkins proxy configuration in Proxy object
+     *
      * @return Proxy object
      */
     public Proxy getJenkinsProxy() {

src/main/java/com/browserstack/automate/ci/common/tracking/PluginsTracker.java

@@ -25,8 +25,8 @@
 public class PluginsTracker {
     private static final MediaType JSON = MediaType.get("application/json; charset=utf-8");
     private static final String URL = "https://api.browserstack.com/ci_plugins/track";
-    private transient OkHttpClient client;
     private final String trackingId;
+    private transient OkHttpClient client;
     private String username;
     private String accessKey;
     private String customProxy;
@@ -77,16 +77,13 @@ private void initializeClient() {
 
         JenkinsProxySettings jenkinsProxy;
         if (customProxy != null) {
-            System.out.println("Custom Proxy In Plugins Tracker: " + customProxy);
             jenkinsProxy = new JenkinsProxySettings(customProxy, null);
         } else {
-            System.out.println("Without Custom Proxy In Plugins Tracker");
             jenkinsProxy = new JenkinsProxySettings(null);
         }
 
         final Proxy proxy = jenkinsProxy.getJenkinsProxy();
         if (proxy != Proxy.NO_PROXY) {
-            System.out.println("Selected some proxy for plugins tracker. " + proxy.toString() + ". And auth: " + jenkinsProxy.hasAuth());
             if (jenkinsProxy.hasAuth()) {
                 final String username = jenkinsProxy.getUsername();
                 final String password = jenkinsProxy.getPassword();

src/main/java/com/browserstack/automate/ci/common/uploader/AppUploader.java

@@ -36,8 +36,6 @@ public String uploadFile()
         }
 
         if (proxy.hasProxy()) {
-            System.out.println("App upload setting proxy for app automate client...");
-            System.out.println(proxy.getHost() + ":" + proxy.getPort() + "," + proxy.getUsername() + ":" + proxy.getPassword());
             appAutomateClient.setProxy(proxy.getHost(), proxy.getPort(), proxy.getUsername(), proxy.getPassword());
         }
         return appAutomateClient.uploadApp(this.appPath).getAppUrl();

src/main/java/com/browserstack/automate/ci/jenkins/BrowserStackReportForBuild.java

@@ -61,7 +61,6 @@ public BrowserStackReportForBuild(final Run<?, ?> build,
         this.projectType = projectType;
         this.logger = logger;
         this.customProxy = customProxy;
-        System.out.println("Initialized BrowserStackReportForBuild with customProxy: " + this.customProxy);
         this.tracker = tracker;
         this.pipelineStatus = pipelineStatus;
         fetchBuildAndSessions();

src/main/java/com/browserstack/automate/ci/jenkins/pipeline/AppUploadStepExecution.java

@@ -40,7 +40,6 @@ protected Void run() throws Exception {
         String customProxy = parentContextEnvVars.get("https_proxy");
         customProxy = Optional.ofNullable(customProxy).orElse(parentContextEnvVars.get("http_proxy"));
 
-        System.out.println("App upload custom proxy: " + customProxy);
         String appId = AppUploaderHelper.uploadApp(run, logger, this.appPath, customProxy);
 
         if (StringUtils.isEmpty(appId)) {

src/main/java/com/browserstack/automate/ci/jenkins/pipeline/BrowserStackPipelineStepExecution.java

@@ -55,8 +55,6 @@ public boolean start() throws Exception {
 
         final PluginsTracker tracker = new PluginsTracker(customProxy);
 
-        System.out.println("\ncustomProxy in pipeline execution: " + customProxy);
-
         BrowserStackCredentials credentials =
                 BrowserStackCredentials.getCredentials(run.getParent(), credentialsId);
 

src/main/java/com/browserstack/automate/ci/jenkins/pipeline/BrowserStackReportStepExecution.java

@@ -38,8 +38,6 @@ protected Void run() throws Exception {
 
         final PluginsTracker tracker = new PluginsTracker(customProxy);
 
-        System.out.println("\ncustomProxy in report generation: " + customProxy);
-
         log(logger, "Generating BrowserStack Test Report via Pipeline for : " + product.name());
 
         String browserStackBuildName = parentContextEnvVars.get(BrowserStackEnvVars.BROWSERSTACK_BUILD_NAME);