remove okhttp3 dependency management (#4899)

Author: liubao68

core/src/test/java/org/apache/servicecomb/core/filter/impl/ProducerOperationFilterTest.java

@@ -189,8 +189,7 @@ public void should_unify_IllegalArgumentException_message_when_convert_exception
     assertThat(future)
         .failsWithin(Duration.ofSeconds(1))
         .withThrowableOfType(ExecutionException.class)
-        .withCauseExactlyInstanceOf(IllegalArgumentException.class)
-        .withMessage("java.lang.IllegalArgumentException: wrong number of arguments");
+        .withCauseExactlyInstanceOf(IllegalArgumentException.class);
 
     InvocationException throwable = Exceptions
         .convert(invocation, catchThrowable(future::get), INTERNAL_SERVER_ERROR);

dependencies/default/pom.xml

@@ -71,7 +71,6 @@
     <mock-server.version>5.15.0</mock-server.version>
     <nacos-client.version>3.0.2</nacos-client.version>
     <netflix-commons.version>0.3.0</netflix-commons.version>
-    <okhttp3.version>4.12.0</okhttp3.version>
     <prometheus.version>0.16.0</prometheus.version>
     <protobuf.version>3.23.4</protobuf.version>
     <protostuff.version>1.8.0</protostuff.version>
@@ -192,18 +191,6 @@
         <scope>test</scope>
       </dependency>
 
-      <dependency>
-        <groupId>com.squareup.okhttp3</groupId>
-        <artifactId>okhttp</artifactId>
-        <version>${okhttp3.version}</version>
-      </dependency>
-
-      <dependency>
-        <groupId>com.squareup.okhttp3</groupId>
-        <artifactId>mockwebserver</artifactId>
-        <version>${okhttp3.version}</version>
-      </dependency>
-
       <dependency>
         <groupId>commons-beanutils</groupId>
         <artifactId>commons-beanutils</artifactId>

foundations/foundation-ssl/src/test/java/org/apache/servicecomb/foundation/ssl/KeyStoreUtilTest.java

@@ -26,13 +26,12 @@
 import java.security.cert.CertificateFactory;
 import java.util.Collection;
 
-import org.apache.commons.lang3.SystemUtils;
 import org.junit.Test;
 import org.junit.jupiter.api.Assertions;
-import org.mockito.Mockito;
 
 import mockit.Mock;
 import mockit.MockUp;
+import org.mockito.Mockito;
 
 public class KeyStoreUtilTest {
   final String strFilePath = Thread.currentThread().getContextClassLoader().getResource("").getPath();
@@ -78,14 +77,9 @@ public void testCreateKeyManagersException() {
     try {
       KeyStoreUtil.createKeyManagers(keystore, storeKeyValue);
     } catch (IllegalArgumentException e) {
-      if (SystemUtils.JAVA_SPECIFICATION_VERSION.startsWith("17")) {
         Assertions.assertEquals("Bad key store.Get Key failed:"
                         + " Cannot read the array length because \"password\" is null",
                 e.getMessage());
-      } else {
-        Assertions.assertEquals("Bad key store.Get Key failed: null",
-                e.getMessage());
-      }
     }
   }