1. Forum
  2. DOVE-Net
  3. Synchronet Programming

  • DDFileLister: Sanitize file descriptions to remove control characters

    From Rob Swindell@VERT to GitLab note in main/sbbs on Wednesday, December 24, 2025 20:24:20
    https://gitlab.synchro.net/main/sbbs/-/merge_requests/590#note_7989

    You seem to be reinventing the wheel here. See `strip_ctrl()` in the Synchronet global JS object.

    ---
    þ Synchronet þ Vertrauen þ Home of Synchronet þ [vert/cvs/bbs].synchro.net
  • From Eric Oulashin@VERT to GitLab note in main/sbbs on Wednesday, December 24, 2025 20:46:06
    https://gitlab.synchro.net/main/sbbs/-/merge_requests/590#note_7990

    But strip_ctrl() also removes Ctrl-A characters. I don't want to remove any Synchronet attribute/color codes in case a file description has any.

    ---
    þ Synchronet þ Vertrauen þ Home of Synchronet þ [vert/cvs/bbs].synchro.net
  • From Rob Swindell@VERT to GitLab note in main/sbbs on Thursday, December 25, 2025 01:34:44
    https://gitlab.synchro.net/main/sbbs/-/merge_requests/590#note_7992

    Okay, that is true. It's a little strange that you call String.replace() repeatedly rather than using a regexp that replaces all occurrences of either problematic character.

    In case, I'm fine to approve for merge.

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