PLUS-POSTECH / soma

Cross-platform CTF problem container manager
Apache License 2.0
24 stars 3 forks source link

Correctly handle OsString #44

Closed Qwaz closed 5 years ago

Qwaz commented 5 years ago

Currently, Soma mixes OsString and String in its internal code. This prevents using non-unicode file name even though host OS supports it.

Edit: non-UTF-8 to non-unicode

KSAlpha commented 5 years ago

Soma implementations reject non-unicode file name only when building docker images.

As non-unicode filename is a very rare use case, I would like to mark this as wontfix. Also, it seems to be a very hard problem to fix.

Supporting non-unicode filename is hard to address because:

However, we would like to provide some workarounds later by supporting user-defined script, which will be executed inside the container in build time. And, by renaming some files using the script, non-unicode filenames can be present inside the image.