Anders Kaseorg <notifications@github.com> wrote:

>We are humans. Being humans, we don’t care how the numbers representing
>characters compare to each other. We care about the properties of
>characters. We should just strip out `[[:cntrl:]]` and leave it at
>that, like we do in other places.
>
>(I don’t see a good reason to strip out spaces, or to start trying to
>decide which punctuation characters are okay. And a blacklist seems
>better than a whitelist because Unicode gets extended.)

I agree, with one exception: it really is necessary to replace '/', because it is not legal in filenames, and because not replacing it would allow one to craft a message which gets logged to more or less arbitrary places in the filesystem.

-- Jeff