-
Notifications
You must be signed in to change notification settings - Fork 54
Bad filename encoding breaking rom list view #39
Comments
Ok i can't reproduce the bug, it's not an unicode problem because it works nice even with filename written in Kana and probably with every valid unicode (symbols, Cyrillic, etc..). What i think it's than the user has uploaded files (at least one) with corrupted encoding filename. Until i can reproduce it, i will just put an exception safe block to simply ignore corrupted filename so it does not affect the list process. Hey @neolao @pasnox @digitalLumberjack @MikaXII @rockaddicted could you help me and try to reproduce it ? Thanks |
and #46; * Granted 'bin' extension for roms on Atari2600 system; * Updated basic pip requirement file to fix Django version on 1.8.x for now; * Fixed Makefile action 'install' to use the right pip requirement file path; * Added a global page footer containing the manager version, close #45; * Only display supported extensions for knowed rom systems from manifest, for unknow system ddirs don't filter on file extension, close #40; * Unmount '/admin/' because it's useless and may give back some ressources; * Naive fix to avoid throwing exception on bad encoded filename in rom list view, just ignore it and continue. Related to #39; * Bump to 1.1.3;
Partly easy - mount a ntfs iso latin file system as utf8 on linux. Should
|
@pasnox oh thanks for the info. This is pretty difficult for me to reproduce this since i dont have any ntfs nor latin file system FS |
Ok, well for now i will consider this as a "wont fix", will notify about in the README and keeping this bug issue open until 1.2.0, and will close it if not any news |
import sys |
@tlanks i dont really understand what you mean, just know that nowadays we are almost full unicode, and utf-8 is mostly used for output formatting. All in all, your code wont change anything even if i was able to reproduce the bug on my recalbox. |
Seems at least on the rom list view, but probably on bios too.
The text was updated successfully, but these errors were encountered: