The current node action was not saved/restored when being forced into node-to-node private chat (by a sysop). This could lead to subsequent chat attempts by the same node to lead to a false indication that the nodes were chatting with each other.
As reported by Altere (ATHEL).

When a sysop wants to chat, he wants to chat right now, damnit!

automatically revert to Raw/TCP mode. This resolves the issue of users that
connect using modem/serial emulators over TCP that do not actually support Telnet and then either try to transfer files, which usually will not work due to escaped (or unescaped) IAC (0xFF) characters, or the user may be sent Telnet Go Ahead (GA) sequences which may display as periodic garbage in their terminal program.

This seemed like a better solution than having (yet another) dedicated terminal server port for "raw" connections over TCP, though a "raw" port still might not be a bad idea for a future enhancement. RLogin is already super close to "raw", so it's probably not really necessary to have another port configured/listening.

- nodelist (default: "nodelist")
- who's online (default: "nodelist -active")
- private message (default: "privatemsg")

This allows the nodelist and private message modules to be "installed" without
over-riding global hot key events (SCFG->External Programs->Global Hot Key
Events), allows the Baja functions NODELIST_ALL, NODELIST_USERS, and
PRIVATE_MESSAGE and JS bbs.list_nodes(), bbs.whos_online(), and
bbs.private_message() methods to use external modules (JS or Baja)
automatically.

The old C++ logic remains for these features if the module's are cleared (set
to a blank string) in SCFG, but at some point, that code will likely be removed.

optionally including the current (your) node.
New @-codes:
"ANODES" (active-nodes count), aliases: "ANODE" and "AN"
"ONODES" (other-active-nodes count), aliases: "ONODE" and "ON"

Make "NN" an alias for "NODE" and "TN" and alias for "TNODE".
Add "TNODES" alias for "TNODE" as well, for symmetry.

the current column number. Use the correct member variable name (column).

record in the node.dab - only the node(s) that have the requested user
number.
In sbbs_t::getsmsg() and ::getsmsg(), strip all invalid Ctrl-A attribute
codes (e.g. clear screen, pause, etc.) since short-messages (telegrams)
can come from external sources.

getsmsg() is now based on the current console column (!=0), not the current
node action.

When using the MSVC2017 CRTL, an unlock() of a region that was not previously
locked is a blocking call causing major slowness in the MSVC2017 build.

lock across things with timeouts in them, but since if another thread has
the file locked we shouldn't be fillding with it anyway, this should be
fine.

Fixes weird (and relatively rare) timing issues with node.dab accesses on a
many core host.

after a user (has been) disconnected and before the node transitions to waiting
for connection (WFC) status. Usually the duration of this status is very short,
but it can be longer (e.g. for running log-out module and event) and for (new)
msg-scan pointer fix-ups.
Updated logout() to remove some cruft and add some more log/debug output.
Automatic new-scan pointer fix-ups when performing new-message scans (if the
current pointer is greater than the last message number, set it to the last
message number) and when saving message pointers.
Passing user_t* to get/putmsgptrs() instead of message number now (to better
detect Guest account).

all exported functions and data types using a 32-bit time_t (time32_t).

longer (use lprintf(LOG_ERR,...) instead, which does).
Added sbbs_t::hacklog() for convenience.

descriptor (-1), this is part of the node.dab error recovery scheme.

allows recovery (e.g. from a shared drive being un/remounted).

inserting (long) typedefs to prevent warnings, but in the case of the JS File()
class, actually supporting file lengths > 32-bit.

the reset daily stats back to the user.dat file ("get" implies read-only).
The resetdailyuserdat() function now optionally doesn't write the reset values.
Functions to that modify the user.dat (putuserdat, putuserrec, and
adjustuserrec) now call a common function to notify nodes of externally
modified user data (with the NODE_UDAT flag) and the current node *is* notified
of external changes (e.g. allowing ;UEDIT changes to the useron to take effect).

Don't prompt for "delete mail from everyone" when there is only one e-mail
waiting for the user.
Use some existing text.dat entries (some old and some recently added) where
identical static strings were still being used for no apparent reason.
Added several text.dat entries for the QWK settings menu, node status, and
some other single-line messages (replacing static strings).

Added HOST and IP keywords to allow restricted access/privileges to/for
specific remote hostnames or IP addresses (wildcards allowed).
All string-argument type ARS keywords (e.g. SHELL, PROT, etc.) now support .can
style wildcards.
The current remote client is now used for protocol, host, and IP ARS checking,
when available, so this requires passing the client pointer around (which
explains why so many files are touched by this change) and takes care of a
long standing to-do item (the user's 'modem' value was used for the PROT
value checking, which was not always correct).

at the start of the function rather than at the end.

Also, for sbbs_t.getsmsg() clear the node.misc bits for all nodes with
the to user rather than just the current node.

node messages and timeleft warnings.

Requested by xvlxxix for a JS game.

RLogin and SSH.

when record can't be locked and thisnode.action != action.

permanently locked under some circumstances on Win32.

getnodedat() now logs "ERR_LOCK" when appropriate (debug help).

are used by OS).
getnodedat and getnodeext() now unlocks if there's a read error and must
retry lock/read.