Don't use multi-release jar for websocket code

6 files changed, 6 insertions, 163 deletions

diff --git a/spark-common/src/main/java/me/lucko/spark/common/SparkPlatform.java b/spark-common/src/main/java/me/lucko/spark/common/SparkPlatform.java
index 61c6062..f609719 100644
--- a/spark-common/src/main/java/me/lucko/spark/common/SparkPlatform.java
+++ b/spark-common/src/main/java/me/lucko/spark/common/SparkPlatform.java
@@ -23,6 +23,8 @@ package me.lucko.spark.common;
 import com.google.common.collect.ImmutableList;
 import com.google.common.collect.ImmutableMap;
 
+import me.lucko.bytesocks.client.BytesocksClient;
+import me.lucko.bytesocks.client.BytesocksClientFactory;
 import me.lucko.spark.common.activitylog.ActivityLog;
 import me.lucko.spark.common.api.SparkApi;
 import me.lucko.spark.common.command.Arguments;
@@ -53,7 +55,6 @@ import me.lucko.spark.common.tick.TickReporter;
 import me.lucko.spark.common.util.BytebinClient;
 import me.lucko.spark.common.util.Configuration;
 import me.lucko.spark.common.util.TemporaryFiles;
-import me.lucko.spark.common.util.ws.BytesocksClient;
 import me.lucko.spark.common.ws.TrustedKeyStore;
 
 import net.kyori.adventure.text.Component;
@@ -126,7 +127,7 @@ public class SparkPlatform {
         String bytesocksHost = this.configuration.getString("bytesocksHost", "spark-usersockets.lucko.me");
 
         this.bytebinClient = new BytebinClient(bytebinUrl, "spark-plugin");
-        this.bytesocksClient = BytesocksClient.create(bytesocksHost, "spark-plugin");
+        this.bytesocksClient = BytesocksClientFactory.newClient(bytesocksHost, "spark-plugin");
         this.trustedKeyStore = new TrustedKeyStore(this.configuration);
 
         this.disableResponseBroadcast = this.configuration.getBoolean("disableResponseBroadcast", false);
diff --git a/spark-common/src/main/java/me/lucko/spark/common/command/modules/SamplerModule.java b/spark-common/src/main/java/me/lucko/spark/common/command/modules/SamplerModule.java
index 049c817..27e790f 100644
--- a/spark-common/src/main/java/me/lucko/spark/common/command/modules/SamplerModule.java
+++ b/spark-common/src/main/java/me/lucko/spark/common/command/modules/SamplerModule.java
@@ -22,6 +22,7 @@ package me.lucko.spark.common.command.modules;
 
 import com.google.common.collect.Iterables;
 
+import me.lucko.bytesocks.client.BytesocksClient;
 import me.lucko.spark.common.SparkPlatform;
 import me.lucko.spark.common.activitylog.Activity;
 import me.lucko.spark.common.command.Arguments;
@@ -43,7 +44,6 @@ import me.lucko.spark.common.tick.TickHook;
 import me.lucko.spark.common.util.FormatUtil;
 import me.lucko.spark.common.util.MediaTypes;
 import me.lucko.spark.common.util.MethodDisambiguator;
-import me.lucko.spark.common.util.ws.BytesocksClient;
 import me.lucko.spark.common.ws.ViewerSocket;
 import me.lucko.spark.proto.SparkSamplerProtos;
 
diff --git a/spark-common/src/main/java/me/lucko/spark/common/util/ws/BytesocksClient.java b/spark-common/src/main/java/me/lucko/spark/common/util/ws/BytesocksClient.java
deleted file mode 100644
index 1db7a67..0000000
--- a/spark-common/src/main/java/me/lucko/spark/common/util/ws/BytesocksClient.java
+++ /dev/null
@@ -1,118 +0,0 @@
-/*
- * This file is part of spark.
- *
- *  Copyright (c) lucko (Luck) <luck@lucko.me>
- *  Copyright (c) contributors
- *
- *  This program is free software: you can redistribute it and/or modify
- *  it under the terms of the GNU General Public License as published by
- *  the Free Software Foundation, either version 3 of the License, or
- *  (at your option) any later version.
- *
- *  This program is distributed in the hope that it will be useful,
- *  but WITHOUT ANY WARRANTY; without even the implied warranty of
- *  MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
- *  GNU General Public License for more details.
- *
- *  You should have received a copy of the GNU General Public License
- *  along with this program.  If not, see <http://www.gnu.org/licenses/>.
- */
-
-package me.lucko.spark.common.util.ws;
-
-import java.util.concurrent.CompletableFuture;
-
-/**
- * A client that can interact with bytesocks.
- *
- * @see <a href="https://github.com/lucko/bytesocks">https://github.com/lucko/bytesocks</a>
- */
-public interface BytesocksClient {
-
-    /**
-     * Creates a new {@link BytesocksClient}.
-     *
-     * <p>Returns {@code null} on Java versions before 11.</p>
-     *
-     * @param host the host
-     * @param userAgent the user agent
-     * @return the client
-     */
-    static BytesocksClient create(String host, String userAgent) {
-        try {
-            return new BytesocksClientImpl(host, userAgent);
-        } catch (UnsupportedOperationException e) {
-            return null;
-        }
-    }
-
-    /**
-     * Creates a new bytesocks channel and returns a socket connected to it.
-     *
-     * @param listener the listener
-     * @return the socket
-     * @throws Exception if something goes wrong
-     */
-    Socket createAndConnect(Listener listener) throws Exception;
-
-    /**
-     * Connects to an existing bytesocks channel.
-     *
-     * @param channelId the channel id
-     * @param listener the listener
-     * @return the socket
-     * @throws Exception if something goes wrong
-     */
-    Socket connect(String channelId, Listener listener) throws Exception;
-
-    /**
-     * A socket connected to a bytesocks channel.
-     */
-    interface Socket {
-
-        /**
-         * Gets the id of the connected channel.
-         *
-         * @return the id of the channel
-         */
-        String getChannelId();
-
-        /**
-         * Gets if the socket is open.
-         *
-         * @return true if the socket is open
-         */
-        boolean isOpen();
-
-        /**
-         * Sends a message to the channel using the socket.
-         *
-         * @param msg the message to send
-         * @return a future to encapsulate the progress of sending the message
-         */
-        CompletableFuture<?> send(CharSequence msg);
-
-        /**
-         * Closes the socket.
-         *
-         * @param statusCode the status code
-         * @param reason the reason
-         */
-        void close(int statusCode, String reason);
-    }
-
-    /**
-     * Socket listener
-     */
-    interface Listener {
-
-        default void onOpen() {}
-
-        default void onError(Throwable error) {}
-
-        default void onText(CharSequence data) {}
-
-        default void onClose(int statusCode, String reason) {}
-    }
-
-}
diff --git a/spark-common/src/main/java/me/lucko/spark/common/util/ws/BytesocksClientImpl.java b/spark-common/src/main/java/me/lucko/spark/common/util/ws/BytesocksClientImpl.java
deleted file mode 100644
index cf1489c..0000000
--- a/spark-common/src/main/java/me/lucko/spark/common/util/ws/BytesocksClientImpl.java
+++ /dev/null
@@ -1,40 +0,0 @@
-/*
- * This file is part of spark.
- *
- *  Copyright (c) lucko (Luck) <luck@lucko.me>
- *  Copyright (c) contributors
- *
- *  This program is free software: you can redistribute it and/or modify
- *  it under the terms of the GNU General Public License as published by
- *  the Free Software Foundation, either version 3 of the License, or
- *  (at your option) any later version.
- *
- *  This program is distributed in the hope that it will be useful,
- *  but WITHOUT ANY WARRANTY; without even the implied warranty of
- *  MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
- *  GNU General Public License for more details.
- *
- *  You should have received a copy of the GNU General Public License
- *  along with this program.  If not, see <http://www.gnu.org/licenses/>.
- */
-
-package me.lucko.spark.common.util.ws;
-
-// Overridden by java 11 source set
-
-class BytesocksClientImpl implements BytesocksClient {
-
-    BytesocksClientImpl(String host, String userAgent) {
-        throw new UnsupportedOperationException();
-    }
-
-    @Override
-    public Socket createAndConnect(Listener listener) throws Exception {
-        throw new UnsupportedOperationException();
-    }
-
-    @Override
-    public Socket connect(String channelId, Listener listener) throws Exception {
-        throw new UnsupportedOperationException();
-    }
-}
diff --git a/spark-common/src/main/java/me/lucko/spark/common/ws/ViewerSocket.java b/spark-common/src/main/java/me/lucko/spark/common/ws/ViewerSocket.java
index 5c7e08c..6a9c2b7 100644
--- a/spark-common/src/main/java/me/lucko/spark/common/ws/ViewerSocket.java
+++ b/spark-common/src/main/java/me/lucko/spark/common/ws/ViewerSocket.java
@@ -22,12 +22,12 @@ package me.lucko.spark.common.ws;
 
 import com.google.protobuf.ByteString;
 
+import me.lucko.bytesocks.client.BytesocksClient;
 import me.lucko.spark.common.SparkPlatform;
 import me.lucko.spark.common.sampler.AbstractSampler;
 import me.lucko.spark.common.sampler.Sampler;
 import me.lucko.spark.common.sampler.window.ProfilingWindowUtils;
 import me.lucko.spark.common.util.MediaTypes;
-import me.lucko.spark.common.util.ws.BytesocksClient;
 import me.lucko.spark.proto.SparkProtos;
 import me.lucko.spark.proto.SparkSamplerProtos;
 import me.lucko.spark.proto.SparkWebSocketProtos.ClientConnect;
diff --git a/spark-common/src/main/java/me/lucko/spark/common/ws/ViewerSocketConnection.java b/spark-common/src/main/java/me/lucko/spark/common/ws/ViewerSocketConnection.java
index f870cb7..9079860 100644
--- a/spark-common/src/main/java/me/lucko/spark/common/ws/ViewerSocketConnection.java
+++ b/spark-common/src/main/java/me/lucko/spark/common/ws/ViewerSocketConnection.java
@@ -22,8 +22,8 @@ package me.lucko.spark.common.ws;
 
 import com.google.protobuf.ByteString;
 
+import me.lucko.bytesocks.client.BytesocksClient;
 import me.lucko.spark.common.SparkPlatform;
-import me.lucko.spark.common.util.ws.BytesocksClient;
 import me.lucko.spark.proto.SparkWebSocketProtos.PacketWrapper;
 import me.lucko.spark.proto.SparkWebSocketProtos.RawPacket;