Merge pull request #90 from browserstack/revert_release_1_1_5

Revert version 1.1.5

Author: yashdsaraf

README.md

@@ -11,7 +11,7 @@ Add this dependency to your project's POM:
 <dependency>
     <groupId>com.browserstack</groupId>
     <artifactId>browserstack-local-java</artifactId>
-    <version>1.1.5</version>
+    <version>1.1.4</version>
 </dependency>
 ```
 

pom.xml

@@ -3,7 +3,7 @@
     <groupId>com.browserstack</groupId>
     <artifactId>browserstack-local-java</artifactId>
     <packaging>jar</packaging>
-    <version>1.1.5</version>
+    <version>1.1.4-SNAPSHOT</version>
 
     <name>browserstack-local-java</name>
     <description>Java bindings for BrowserStack Local</description>

src/main/java/com/browserstack/local/Local.java

@@ -23,7 +23,7 @@ public class Local {
     private LocalProcess proc = null;
 
     // Current version of binding package, used for --source option of binary
-    private static final String packageVersion = "1.1.5";
+    private final String packageVersion = "1.1.4";
     private final Map<String, String> parameters;
     private final Map<String, String> avoidValueParameters;
 
@@ -130,15 +130,6 @@ public boolean isRunning() throws Exception {
         return isProcessRunning(pid);
     }
 
-    /**
-     * Returns the package version
-     *
-     * @return {String} package version
-     */
-    public static String getPackageVersion() {
-        return packageVersion;
-    }
-
     /**
      * Creates a list of command-line arguments for the Local instance
      *

src/main/java/com/browserstack/local/LocalBinary.java

@@ -2,15 +2,11 @@
 
 import org.apache.commons.io.FileUtils;
 import java.io.IOException;
-import java.io.InputStream;
 import java.io.BufferedReader;
 import java.io.InputStreamReader;
 import java.io.File;
 import java.net.URL;
-import java.net.URLConnection;
 import java.util.regex.Pattern;
-import java.util.zip.GZIPInputStream;
-import java.util.zip.ZipException;
 
 class LocalBinary {
 
@@ -62,8 +58,7 @@ private void initialize() throws LocalException {
             throw new LocalException("Failed to detect OS type");
         }
 
-        String sourceURL = BIN_URL;
-        httpPath = sourceURL + binFileName;
+        httpPath = BIN_URL + binFileName;
     }
 
     private boolean isAlpine() {
@@ -179,7 +174,7 @@ private void downloadBinary(String destParentDir, Boolean custom) throws LocalEx
             URL url = new URL(httpPath);
 
             File f = new File(source);
-            newCopyToFile(url, f);
+            FileUtils.copyURLToFile(url, f);
 
             changePermissions(binaryPath);
         } catch (Exception e) {
@@ -197,26 +192,4 @@ private void changePermissions(String path) {
     public String getBinaryPath() {
         return binaryPath;
     }
-
-    private static void newCopyToFile(URL url, File f) throws IOException {
-        URLConnection conn = url.openConnection();
-        conn.setRequestProperty("User-Agent", "browserstack-local-java/" + Local.getPackageVersion());
-        conn.setRequestProperty("Accept-Encoding", "gzip, *");
-        String contentEncoding = conn.getContentEncoding();
-
-        if (contentEncoding == null || !contentEncoding.toLowerCase().contains("gzip")) {
-            FileUtils.copyToFile(conn.getInputStream(), f);
-            return;
-        }
-
-        try (InputStream stream = new GZIPInputStream(conn.getInputStream())) {
-            if (System.getenv().containsKey("BROWSERSTACK_LOCAL_DEBUG_GZIP")) {
-                System.out.println("using gzip in " + conn.getRequestProperty("User-Agent"));
-            }
-
-            FileUtils.copyToFile(stream, f);
-        } catch (ZipException e) {
-            FileUtils.copyURLToFile(url, f);
-        }
-    }
 }