Attempted fix for #488, reorderedlogic to use server url only, added flag to change this behavior to use external host value if set

Author: CyferShepard

README.md

@@ -45,6 +45,7 @@
 | MINIMUM_SECONDS_TO_INCLUDE_PLAYBACK | `1`       | `10`                            | The minimum time (in seconds) to include a playback record, which can be used to exclude short playbacks                                 |
 | IS_EMBY_API                         | `false`   | `true`                          | Set to true if using Emby instead of Jellyfin                                                                                            |
 | JF_USE_WEBSOCKETS                   | `true`    | `false`                         | Enables using Jellyfins websocket connection for session data (No effect on Emby)                                                        |
+| JS_USE_EXTERNAL_HOST                | `false`   | `true`                          | Prioritises using External host value for some internal communications and processes                                                     |
 
 ## Getting Started with Development
 

backend/classes/jellyfin-api.js

@@ -457,8 +457,12 @@ class JellyfinAPI {
         compareVersions(this.version, "10.11.0") >= 0 &&
         (process.env.JF_USE_WEBSOCKETS === undefined || process.env.JF_USE_WEBSOCKETS.toLowerCase() !== "false")
       ) {
-        const socketUrl =
-          (this.config.JF_EXTERNAL_HOST ?? this.config.JF_HOST).replace(/^http/, "ws").replace(/^https/, "wss") + "/socket";
+        const hostUrl =
+          process.env.JS_USE_EXTERNAL_HOST === undefined || process.env.JS_USE_EXTERNAL_HOST.toLowerCase() !== "false"
+            ? this.config.JF_HOST
+            : this.config.JF_EXTERNAL_HOST ?? this.config.JF_HOST;
+
+        const socketUrl = hostUrl.replace(/^http/, "ws").replace(/^https/, "wss") + "/socket";
         const sessionData = await getSessionData(socketUrl, this.config.JF_API_KEY);
         if (sessionData != null) {
           return sessionData;