Low-order non-printable ASCII characters in worksheet name should be escaped or removed about axlsx HOT 5 OPEN

Does it make sense to retain such (escaped) characters in sheet names? In cells for example they are removed, see Cell#clean_value which calls Axlsx.sanitize which removes control characters.

from axlsx.

@noniq I'm not sure which is better, removing or escaping non-printable ASCII characters in sheet names. If it is undocumented or is undefined behavior, it's best to be removed. if it is a valid name, it make sense to retain it.

from axlsx.

My gut feeling is that non-printable characters should not be allowed to be part of a sheet name. Are there any references to what constitutes a valid sheet name in the Excel docs?

from axlsx.

I didn't find any references about what to do with non-printable characters 🤔.
Either removing or escaping is okay to me.

from axlsx.

Would you like to prepare a pull request? axlsx is currently unmaintained though, so please create the pull request in the https://github.com/caxlsx/caxlsx repository (that's the actively maintained community fork).

from axlsx.