Merge pull request #824 from freyacodes/dev

v3.7.4 release

Author: topi314

.github/workflows/release.yml

@@ -18,14 +18,15 @@ jobs:
     runs-on: ubuntu-latest
     steps:
       - name: Checkout
-        uses: actions/checkout@v2
+        uses: actions/checkout@v3
 
       - name: Download Artifacts
-        uses: actions/download-artifact@v2
+        uses: actions/download-artifact@v3
         with:
           name: Lavalink.jar
 
       - name: Upload Artifacts to GitHub Release
-        uses: softprops/action-gh-release@v1
+        uses: ncipollo/release-action@v1
         with:
-          files: Lavalink.jar
+          artifacts: Lavalink.jar
+          allowUpdates: true

CHANGELOG.md

@@ -3,6 +3,9 @@
 Each release usually includes various fixes and improvements.
 The most noteworthy of these, as well as any features and breaking changes, are listed here.
 
+## 3.7.4
+* Fix an issue where Lavalink would not destroy a session when a client disconnects
+
 ## 3.7.3
 * Fix breaking change where `/decodetrack` would return a full track instead of the track info
 

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(