fix usage of wrong session id in afterConnectionClosed

topi314 · topi314 · commit cb320d1c11e4 · 2023-02-02T19:11:45.000+01:00
diff --git a/LavalinkServer/src/main/java/lavalink/server/io/SocketServer.kt b/LavalinkServer/src/main/java/lavalink/server/io/SocketServer.kt
@@ -147,7 +147,7 @@ class SocketServer(
     }
 
     override fun afterConnectionClosed(session: WebSocketSession, status: CloseStatus) {
-        val context = contextMap.remove(session.id) ?: return
+        val context = contextMap.remove(session.attributes["sessionId"]) ?: return
         if (context.resumeKey != null) {
             resumableSessions.remove(context.resumeKey!!)?.let { removed ->
                 log.warn(

Original file line number	Diff line number	Diff line change
`@@ -147,7 +147,7 @@ class SocketServer(`
`147`	`147`	`}`
`148`	`148`
`149`	`149`	`override fun afterConnectionClosed(session: WebSocketSession, status: CloseStatus) {`
`150`		`- val context = contextMap.remove(session.id) ?: return`
	`150`	`+ val context = contextMap.remove(session.attributes["sessionId"]) ?: return`
`151`	`151`	`if (context.resumeKey != null) {`
`152`	`152`	`resumableSessions.remove(context.resumeKey!!)?.let { removed ->`
`153`	`153`	`log.warn(`