Allow all characters (except /) when sanitazing

[K1-K1]

No description provided.

[bjesus]

I've been using the roles from https://github.com/parshap/node-sanitize-filename to replace some more unsafe characters, so more like this:

var illegalRe = /[\/\?<>\\:\*\|"]/g;
var controlRe = /[\x00-\x1f\x80-\x9f]/g;
var reservedRe = /^\.+$/;
var windowsReservedRe = /^(con|prn|aux|nul|com[0-9]|lpt[0-9])(\..*)?$/i;
var windowsTrailingRe = /[\. ]+$/;

function sanitize(input, replacement) {
  if (typeof input !== 'string') {
    throw new Error('Input must be string');
  }
return input
    .replace(illegalRe, replacement)
    .replace(controlRe, replacement)
    .replace(reservedRe, replacement)
    .replace(windowsReservedRe, replacement)
    .replace(windowsTrailingRe, replacement);
}

[osnr]

Thanks to both of you! I've decided to still replace spaces but have allowed all the other characters, I think: 922e2bb#diff-bdb5cae59928803bd7b94e819be96e2f3f8dc6e3d15beae20645dec6a188cdc4R38

open to suggestions if there's a compelling use case for keeping spaces. (I also thought about getting rid of the tab ID suffix, but have decided against that for now; it adds some complexity)