Rob Swindell authored 2 months ago and Deucе committed 2 months ago

Kind of a shot in the dark here: Max (WESTLINE) is reporting HEAP CORRUPTION
debug assertion in websrvr.dll. In the 2 instances reported, a long (336
char) JSON "query value" was logged by apparent spam-bot trying to create a
a new user account ("send-me-free-stuff" is one of the JSON properties).

JS_NewStringCopyZ() can return NULL in a low memory situation, though I don't
know that explains possible heap corruption.

This way, when an error occurs, it will at least terminate the
session instead of requiring you to disconnect yourself.

This partially addresses issue 174, assuming it's actually telnets.

Apple GNU make treats unescaped # in variables as starting a comment.
But GNU grep >= v3.8 will spew a warning with an extra backslash there.

GNU grep >= 3.8 complains and it's highly confusing.
Thanks for sticking with it Dan_C.

Presumably, this is broken because I cross-compile.
Also, this is where I'm rewriting history to remove
the debug libs that were over 100M

...But Win64 isn't.  It crashes somewhere in the decoding function,
and I have no idea why... it doesn't seem to be because of the thread
thing.

I assume it's because of the cross-built win64 libs, but I can't really
be sure. :(

So, we can't use threads with these libs. :(
This is going to be a pain to build these due to the dependencies.

This can't happen, and it breaks the thread stuff.

Just statically link it here, I'm not interested in adding another
DLL into the archive, and there's no good reason to NOT support it.

An alternative to using PPM, relies on libjxl from the JPEG.
JPEG XL is a superior modern format that supports both lossy and
lossless compression and does an excellent job at both.

The format also supports multiple frames (ie: animation), so can
be used for Cool Stuff in the future.

As an example, a large image that is 130MB in PPM format is only
6.8MB in JPEG XL format using the default quality settings of
ImageMagick (whatever they are).

All the scripts I used this in previously had a read_apc() function.

Commands are added.

This allows us to avoid needing a large number of extra libraries,
and it appears sRGB (ie: what we want) is the default anyway.

These are useful for trouble-shooting/experimenting with file share modes
and region locks, so make them easier to build.

Order of options no longer matters.

This also supports static linking for OSs that don't have sane ways
of sharing libraries system-wide.

With this, my fancy intro is quite usable...

Uses libjxl, makes the video demo more possible.

Does not yet have a feature test sequence, documentation, support
in the gmake build system, runtime linking, etc.  Just a quick
hack.

It also looks like I can parallize the decode, should should also
help things out.

It was badly broken, especially with large files... we now don't
try console.write() unless there's enough space in
console.output_buffer_space.

On my system (debug build of Synchronet), the most I can push through
console.write() over telnet is about 1.9MB/s.  My release build of
SyncTERM can consume about 4MB/s of string data, so Synchronet is the
choke point in my setup here. (SSH is much worse)

My super-cool demo thing ends up needing about 90 seconds to preload
all the cache stuff, so it's simply not useable, even locally. :(

If I use client.socket.send(), I can unlock the Synchronet throughput,
but I can't remember if client.socket is the passthru socket with SSH
or not... and there's no way to synchronize the socket with the console
at the end of the send (console has flush(), Socket doesn't, and even if
it did, it wouldn't flush through to the output buffer).

Check and clear/invalidate node_socket while holding the node.dab record
lock.

This should fix the error reported by kk4qnb (KK4QBN)

DDFileLister: New configuration setting: useFilenameIfNoDescription - If a description is empty, show the filename in the list instead. For Retro Guy.

See merge request !487

DDFileLister: New configuration setting: useFilenameIfNoDescription - If a description is empty, show the filename in the list instead. For Retro Guy.