From f54b7ffc44f4544ea66f0737b2296df84be998a9 Mon Sep 17 00:00:00 2001 From: Adam Date: Fri, 8 Apr 2022 10:58:13 -0400 Subject: [PATCH] session manager: open session on executor thread This avoids blocking the main thread on startup during the http call --- .../runelite/client/ClientSessionManager.java | 19 +++++++++++-------- 1 file changed, 11 insertions(+), 8 deletions(-) diff --git a/runelite-client/src/main/java/net/runelite/client/ClientSessionManager.java b/runelite-client/src/main/java/net/runelite/client/ClientSessionManager.java index 3f884b092a..bbc179b3e3 100644 --- a/runelite-client/src/main/java/net/runelite/client/ClientSessionManager.java +++ b/runelite-client/src/main/java/net/runelite/client/ClientSessionManager.java @@ -62,15 +62,18 @@ public class ClientSessionManager public void start() { - try + executorService.execute(() -> { - sessionId = sessionClient.open(); - log.debug("Opened session {}", sessionId); - } - catch (IOException ex) - { - log.warn("error opening session", ex); - } + try + { + sessionId = sessionClient.open(); + log.debug("Opened session {}", sessionId); + } + catch (IOException ex) + { + log.warn("error opening session", ex); + } + }); scheduledFuture = executorService.scheduleWithFixedDelay(RunnableExceptionLogger.wrap(this::ping), 1, 10, TimeUnit.MINUTES); }