cleanup: remove some dependencies that are easy to replace (#4408)

Author: jkwatson-verta

backend/common/pom.xml

@@ -202,10 +202,10 @@
             <groupId>software.amazon.awssdk</groupId>
             <artifactId>s3</artifactId>
         </dependency>
-        <!-- https://mvnrepository.com/artifact/com.google.cloud/google-cloud-storage -->
         <dependency>
-            <groupId>com.google.cloud</groupId>
-            <artifactId>google-cloud-storage</artifactId>
+            <groupId>com.google.protobuf</groupId>
+            <artifactId>protobuf-java-util</artifactId>
+            <version>3.25.1</version>
         </dependency>
         <!-- https://mvnrepository.com/artifact/com.h2database/h2 -->
         <dependency>

backend/common/src/main/java/ai/verta/modeldb/common/artifactStore/storageservice/nfs/NFSService.java

@@ -5,7 +5,6 @@
 import ai.verta.modeldb.common.exceptions.InvalidArgumentException;
 import ai.verta.modeldb.common.exceptions.ModelDBException;
 import com.amazonaws.services.s3.model.PartETag;
-import com.google.api.client.util.IOUtils;
 import com.google.rpc.Code;
 import jakarta.servlet.http.HttpServletRequest;
 import java.io.File;
@@ -106,7 +105,7 @@ String storeFile(
       }
       LOGGER.trace("NFSService - storeFile -  file found : {}", foldersExists.getAbsolutePath());
       var fileOutputStream = new FileOutputStream(destinationFile);
-      IOUtils.copy(uploadedFileInputStream, fileOutputStream);
+      uploadedFileInputStream.transferTo(fileOutputStream);
       fileOutputStream.close();
       uploadedFileInputStream.close();
       LOGGER.trace(

backend/common/src/main/java/ai/verta/modeldb/common/artifactStore/storageservice/s3/S3Service.java

@@ -13,7 +13,6 @@
 import com.amazonaws.SdkClientException;
 import com.amazonaws.services.s3.model.*;
 import com.amazonaws.services.s3.transfer.TransferManagerBuilder;
-import com.google.api.client.http.HttpStatusCodes;
 import com.google.rpc.Code;
 import jakarta.servlet.http.HttpServletRequest;
 import java.io.*;
@@ -159,7 +158,7 @@ public void commitMultipart(String s3Key, String uploadId, List<PartETag> partET
           client.getClient().completeMultipartUpload(completeMultipartUploadRequest);
       LOGGER.info("upload result: {}", result);
     } catch (AmazonS3Exception e) {
-      if (e.getStatusCode() == HttpStatusCodes.STATUS_CODE_BAD_REQUEST) {
+      if (e.getStatusCode() == 400) {
         LOGGER.info("message: {} additional details: {}", e.getMessage(), e.getAdditionalDetails());
         throw new ModelDBException(e.getErrorMessage(), Code.FAILED_PRECONDITION);
       }

backend/server/pom.xml

@@ -101,11 +101,6 @@
             <scope>TEST</scope>
         </dependency>
 
-        <!-- https://mvnrepository.com/artifact/com.google.cloud/google-cloud-storage -->
-        <dependency>
-            <groupId>com.google.cloud</groupId>
-            <artifactId>google-cloud-storage</artifactId>
-        </dependency>
         <!-- https://mvnrepository.com/artifact/org.apache.logging.log4j/log4j-slf4j-impl -->
         <dependency>
             <groupId>org.apache.logging.log4j</groupId>

backend/server/src/main/java/ai/verta/modeldb/entities/versioning/RepositoryEntity.java

@@ -18,7 +18,6 @@
 import ai.verta.uac.GetResourcesResponseItem.OwnerTrackingCase;
 import ai.verta.uac.ResourceVisibility;
 import ai.verta.uac.Workspace;
-import com.google.api.client.util.Objects;
 import java.io.Serializable;
 import java.util.*;
 import javax.persistence.*;
@@ -350,6 +349,6 @@ public boolean isDataset() {
   }
 
   public boolean isProtected() {
-    return Objects.equal(repositoryAccessModifier, RepositoryModifierEnum.PROTECTED.ordinal());
+    return Objects.equals(repositoryAccessModifier, RepositoryModifierEnum.PROTECTED.ordinal());
   }
 }

backend/server/src/main/java/ai/verta/modeldb/telemetry/TelemetryCron.java

@@ -8,7 +8,6 @@
 import ai.verta.modeldb.utils.ModelDBHibernateUtil;
 import ai.verta.modeldb.utils.ModelDBUtils;
 import ai.verta.uac.CollectTelemetry;
-import com.google.api.client.http.HttpMethods;
 import com.google.protobuf.Value;
 import java.io.BufferedReader;
 import java.io.FileNotFoundException;
@@ -58,7 +57,7 @@ public void run() {
 
         HttpURLConnection httpClient =
             (HttpURLConnection) new URL(telemetryUtils.getConsumer()).openConnection();
-        httpClient.setRequestMethod(HttpMethods.POST);
+        httpClient.setRequestMethod("POST");
         httpClient.setDoOutput(true);
         httpClient.setRequestProperty("Content-Type", "application/json");
         httpClient.setRequestProperty("Accept", "application/json");

backend/server/src/test/java/ai/verta/modeldb/ArtifactStore/NFSArtifactStoreTest.java

@@ -44,7 +44,6 @@
 import ai.verta.uac.UserInfo;
 import ai.verta.uac.VertaUserInfo;
 import ai.verta.uac.Workspace;
-import com.google.api.client.util.IOUtils;
 import com.google.common.util.concurrent.Futures;
 import io.grpc.Context;
 import io.grpc.ManagedChannel;
@@ -331,7 +330,7 @@ private void loggedArtifactByUrlExperimentRun() throws IOException {
       httpClient.setDoOutput(true);
       httpClient.setRequestProperty("Content-Type", "application/json");
       try (OutputStream out = httpClient.getOutputStream()) {
-        IOUtils.copy(inputStream, out);
+        inputStream.transferTo(out);
         out.flush();
       }
 
@@ -367,7 +366,7 @@ private void loggedArtifactByUrlProject() throws IOException {
       httpClient.setDoOutput(true);
       httpClient.setRequestProperty("Content-Type", "application/json");
       try (OutputStream out = httpClient.getOutputStream()) {
-        IOUtils.copy(inputStream, out);
+        inputStream.transferTo(out);
         out.flush();
       }
 
@@ -400,7 +399,7 @@ public void getArtifactByUrlExperimentRunTest() throws IOException {
     String rootPath = System.getProperty("user.dir");
     FileOutputStream fileOutputStream =
         new FileOutputStream(rootPath + File.separator + artifactKey);
-    IOUtils.copy(inputStream, fileOutputStream);
+    inputStream.transferTo(fileOutputStream);
     fileOutputStream.close();
     inputStream.close();
 
@@ -437,7 +436,7 @@ public void getArtifactByUrlProjectTest() throws IOException {
     String rootPath = System.getProperty("user.dir");
     FileOutputStream fileOutputStream =
         new FileOutputStream(rootPath + File.separator + artifactKey);
-    IOUtils.copy(inputStream, fileOutputStream);
+    inputStream.transferTo(fileOutputStream);
     fileOutputStream.close();
     inputStream.close();
 

pom.xml

@@ -267,25 +267,6 @@
                 <scope>import</scope>
             </dependency>
 
-
-            <!-- https://mvnrepository.com/artifact/com.google.cloud/google-cloud-storage -->
-            <dependency>
-                <groupId>com.google.cloud</groupId>
-                <artifactId>google-cloud-storage</artifactId>
-                <version>2.35.0</version>
-                <exclusions>
-                    <exclusion>
-                        <groupId>io.grpc</groupId>
-                        <artifactId>*</artifactId>
-                    </exclusion>
-                </exclusions>
-            </dependency>
-            <!-- pin this dependency of ^^^ to upgrade for a CVE (https://snyk.io/vuln/SNYK-JAVA-COMGOOGLEOAUTHCLIENT-2807808) -->
-            <dependency>
-                <groupId>com.google.oauth-client</groupId>
-                <artifactId>google-oauth-client</artifactId>
-                <version>1.35.0</version>
-            </dependency>
             <!-- https://mvnrepository.com/artifact/io.prometheus/simpleclient_servlet -->
             <dependency>
                 <groupId>io.prometheus</groupId>