• src/sbbs3/bat_xfer.cpp execfile.cpp file.cpp filedat.c filedat.h ftpsr

    From Rob Swindell (on Debian Linux)@VERT to Git commit to main/sbbs/master on Thu Feb 20 20:29:35 2025
    https://gitlab.synchro.net/main/sbbs/-/commit/80ff3be5a2aaba9de98eb936
    Modified Files:
    src/sbbs3/bat_xfer.cpp execfile.cpp file.cpp filedat.c filedat.h ftpsrvr.c listfile.cpp qwk.cpp upload.cpp userdat.c websrvr.c
    Log Message:
    Add optional error/result outparam to highlevel filedat API functions

    - loadfile
    - addfile
    - removefile
    - updatefile

    As discovered via ftp server "error updating" file log message, no details could be logged since these functions just returned a simple bool. So, we
    now support passing an optional output parameter to capture the result/error value to be displayed or logged when these functions return false.

    Unrelated fixes to start_batch_download():
    When passing a list of files on the command to the file transfer protocol driver (e.g. sz), if any filename contains spaces, that filename must be
    quoted or else sz will fail with error 'sz: can read only one file from stdin'. Also, the trailing whitespace from the list of filenames must be removed or else sz will report errors 'sz: cannot open : No such file or directory'
    and 'sz: Transfer incomplete'.

    ---
    þ Synchronet þ Vertrauen þ Home of Synchronet þ [vert/cvs/bbs].synchro.net