I wanted a couple of features for filters (e.g. blocked IP addreses):
- expiration date, optionally (only block for a limited time)
- display details of reason for filter in log messages (at time of block)

I've been thinking for some time that these files should be converted to .ini,
but then it occurred to me that I could do a sort of hybrid where the metadata
was key/value pairs tab-delimited from the search-pattern (first text on the
line). This is backward comaptible, relatively easy to view/edit by hand, and
extensible (easy to add/remove metadata fields in the future). So they'll
remain as text/*.can files.

The maximum line length for findstr/trashcan parsed files extended from 255 to
1000 characters.

Moved all trashcan-related functions from scfglib1.c to (new file) trash.c.

For the first time, I actually tested a build on WSL *before* committing to
Git. So, this *should* pass CI the first time. :-O

That doesn't trigger a Coverity defect this time (CID 470457)

resulting from commit 5f1c39f0

Fix for issue #678

With 0xE0 being used for ciolib "super-extended scancodes", a literal
0xe0 can't pass through the input path.  This is an issue in CP866 (р)
and KOI8-U (Ю) as well as CP437 α.

Should fix SyncTERM SF bug 123.

Only the common/UIFC-type arguments should be passed-on.

This fixes issue #646

It also appeared that the user editor spawning wouldn't work correctly
when arguments were passed to umonitor (missing a space).

Mostly (all?) about unchecked return values. Unexpected failures to read or
write some files could've definitely led to some weird bugs.

A sysop can, and may want to, chat with a user before they succesfully login.

On the node options popup menu, these options weren't visible/available unless
there was a user logged-in, but they're usable and useful even when there
is not a user logged-in.

I think this is what nelgin/Keyop was asking for via IRC.

Also a bit more constification and cleared up the legacy sbbs_get_ini_fname()
usage: the hostname argument hadn't be used in a long, long time.

Needed for text.dat support in SBBSecho

load_cfg really should be made either a static library or an exported
function from libsbbs.so that everything reuses. Kind of silly to be
recompiling and linking these files with every app now.

CID 174223
CID 174227
CID 174461
CID 174519
CID 229602
CID 319040
CID 319137
CID 319159

While we're doing that, actually use it in SyncTERM so the offline
scrollback retains fonts.

Also, fix an issue where the current mode would be updated with
the custom parameters when you changed them.

Finally, remove some unused variables.

Don't return an error if the node#/node.cnf file can't be opened for all uses of load_cfg() except from the terminal server. This fixes #214 for Tracker1

Fix issue reported WitNik (BGGRSCYN) on DOVE-Net:
```
sbbs@raspberrypi:~ $ /sbbs/exec/umonitor -iC
...
sh: 1: /sbbs/exec/syncterm-iC: not found
```

when selecting a node (data comes from node#/client.ini). For Nelgin.

Was using <data_dir>/temp to check for free space. data/temp is
not a normally created directory, so usually this check would fail
and just report 0 bytes free space. Instead, lets check the data
directory itself.

Combined with a fix in uifc32.c, this fixes some problems with spawning
external programs (e.g. to edit files, run scfg, echocfg, etc.) and
when returning having a different keyboard input functionality and the
screen all fubarred. Tested with Linux/ncurses.

Added an "All Servers" option.
Fixed the "Telnet server" (now "Terminal Server") to set the correct/modern semaphore file.
Simplified the implementation of this function.

(e.g. logon, new-user, quiet mode) - these are value node status values
with a user on the node.

Also address GCC warnings about unsafe sprintf usage, e.g.
warning: ‘%s’ directive writing up to 79 bytes into a region of size 59

rather than just falling over and twitching like an injured slug

If jsexec is used for a CGI script and the SBBSCTRL env var was not
set, a warning would be printed (to stderr), even if the -c option
was used to specify the path to the ctrl dir.

get_ctrl_dir() now accepts a boolean argument to specify whether to
print a warning when the default value ("/sbbs/ctrl") is used.
Pass FALSE for jsexec, TRUE for other callers (currently).

Removed some unnecessary carriage returns from printf statements.

Moved twitlist.cfg and spamblock.cfg to "Edit Filter File" menu.

Updated Menu Item names (e.g. "Configure BBS" rather "Run SCFG").
Added menu options to view echostats.ini and badareas.lst

Fixed capitalization and description of "SBBSecho".

stack. Use the heap instead.

As reported by Ragnarok (DOCKSUD)

not set (or blank), use the default ("/sbbs/ctrl") and print a warning to
stderr. This resolves a FAQ for sysops that struggle with setting the SBBSCTRL
env var, but do have sbbs install in /sbbs or a symlink from /sbbs to where
sbbs is actually installed.

Note: this function is different than the one previously defined in getctrl.c
but never used. If/when we ever really need to support PREFIX, then perhaps
we'll use that one (which copies a string to a buffer).