From 695bb40b17aed553f1c147fa1fa7629929c0afe0 Mon Sep 17 00:00:00 2001
From: Bryan Ashby <bryan@l33t.codes>
Date: Wed, 31 Aug 2022 11:47:03 -0600
Subject: [PATCH] More improved logging around client changes

---
 core/client_term.js | 10 +++++-----
 core/connect.js     |  2 +-
 2 files changed, 6 insertions(+), 6 deletions(-)

diff --git a/core/client_term.js b/core/client_term.js
index 938442fc..a5b3890a 100644
--- a/core/client_term.js
+++ b/core/client_term.js
@@ -60,6 +60,11 @@ function ClientTerminal(output) {
         set: function (ttype) {
             termType = ttype.toLowerCase();
 
+            Log.debug(
+                { encoding: this.outputEncoding },
+                `Terminal type changed to ${termType}; Adjusting output encoding`
+            );
+
             if (this.isNixTerm()) {
                 this.outputEncoding = 'utf8';
             } else {
@@ -69,11 +74,6 @@ function ClientTerminal(output) {
             //  :TODO: according to this: http://mud-dev.wikidot.com/article:telnet-client-identification
             //  Windows telnet will send "VTNT". If so, set termClient='windows'
             //  there are some others on the page as well
-
-            Log.debug(
-                { encoding: this.outputEncoding },
-                'Set output encoding due to terminal type change'
-            );
         },
     });
 
diff --git a/core/connect.js b/core/connect.js
index 71b41582..affa4eec 100644
--- a/core/connect.js
+++ b/core/connect.js
@@ -157,7 +157,7 @@ const ansiQuerySyncTermFontSupport = (client, cb) => {
             const [_, w] = pos;
             if (w === 1) {
                 // cursor didn't move
-                client.log.info('Enabling SyncTERM font support');
+                client.log.info(`SyncTERM font support enabled on node ${client.node}`);
                 client.term.syncTermFontsEnabled = true;
             }
         },