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

Require that files are valid UTF-8 #308

Open
jdforrester opened this issue Jul 13, 2020 · 2 comments
Open

Require that files are valid UTF-8 #308

jdforrester opened this issue Jul 13, 2020 · 2 comments

Comments

@jdforrester
Copy link
Member

Not sure if we'd want this for all files, or just JSON files, but…

No obvious upstream rule for this (just the BOM one).

@edg2s
Copy link
Member

edg2s commented Jul 13, 2020

Specifically JSON with "invalid" UTF-8 can cause PHP json_decode to throw, which is an MW-specific issue.

@edg2s
Copy link
Member

edg2s commented Jul 16, 2020

I'm not sure how we would implement a test in JS that PHP json_decode will throw.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

No branches or pull requests

2 participants