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.

Sign up for GitHub

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

Jump to bottom

Server returns 500 on C(++) compilation error #129

Closed
GeorgeBekh opened this issue Dec 12, 2019 · 2 comments
Closed

Server returns 500 on C(++) compilation error #129

GeorgeBekh opened this issue Dec 12, 2019 · 2 comments

Comments

@GeorgeBekh
Copy link

GeorgeBekh commented Dec 12, 2019 •
edited
Loading

Right now when you submit some invalid C code the server responds with 500. You can check it with official demo https://api.judge0.com/dummy-client.html

This happens because gcc compilation message contains non UTF-8 characters.

This PR handles such characters and fixes the error - #128
@GeorgeBekh GeorgeBekh changed the title Server returns 500 error on compilation error Server returns 500 on C(++) compilation error Dec 12, 2019
@GeorgeBekh GeorgeBekh mentioned this issue Dec 12, 2019
Closed
hermanzdosilovic added a commit that referenced this issue Dec 31, 2019
@hermanzdosilovic
Respond with error message when some submission attributes cannot be …
cdcaa51 
…converted to UTF-8 (solves #129)
@hermanzdosilovic
Copy link
Member

I have pushed commit with the changes that do the best thing we can currently do to notify user what is the problem. As commented on your PR force_encoding("UTF-8") does not solve the problem.

@hermanzdosilovic
Copy link
Member

This is now resolved with Judge0 API v1.5.0.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Fix 500 on C compilation error GeorgeBekh/api
2 participants
@hermanzdosilovic @GeorgeBekh