Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Non-UTF8 JSON files fail to deserialize #764

Closed
2 tasks done
VisualRedux opened this issue Jul 30, 2023 · 3 comments · Fixed by #858
Closed
2 tasks done

Non-UTF8 JSON files fail to deserialize #764

VisualRedux opened this issue Jul 30, 2023 · 3 comments · Fixed by #858
Labels
bug Something isn't working

Comments

@VisualRedux
Copy link

Checklist

Edition

Windows GUI Application

Describe your issue here

As soon as I press render I get this message: ERROR: '0xFF' is an invalid start of a value. LineNumber: 0 | BytePositionInLine: 0.

Any suggestions?

Add any related files or extra information here

Here is the JSON file: https://drive.google.com/file/d/1VqEx_lW7-dSUBwpoJbYg0U6Pt6xIitm5/view

@VisualRedux VisualRedux added the bug Something isn't working label Jul 30, 2023
@ScrubN
Copy link
Collaborator

ScrubN commented Jul 31, 2023

For some strange reason, your chat starts with 2 invalid bytes (FF FE). You can open it with a hex editor or use a hex editor plugin in a text editor like vscode to delete those bytes and save to a new file.
image

@ScrubN ScrubN closed this as not planned Won't fix, can't repro, duplicate, stale Jul 31, 2023
@ScrubN
Copy link
Collaborator

ScrubN commented Jul 31, 2023

I just realized, those bytes are because the document was written with UTF16. You should be able to convert it to UTF8 with a text editor like vscode or notepad++

@ScrubN
Copy link
Collaborator

ScrubN commented Oct 15, 2023

The user should not be expected to know the differences between UTF8 and UTF16 nor fix problems caused by the differences.

@ScrubN ScrubN reopened this Oct 15, 2023
@ScrubN ScrubN changed the title Chat render error Non-UTF8 JSON files fail to deserialize Oct 15, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants