-
Notifications
You must be signed in to change notification settings - Fork 70
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
Replace JDoodle with Piston #20
Replace JDoodle with Piston #20
Comments
Hi @Rishabh-malhotraa! Can I work on this for hacktoberfest? |
Sure go right ahead. Here is a bit more context about this issue. So for us to run code on the browser, we make an API call to a remote code execution service, right now we use something called jdoodle, it only provides us with 200 execution per day. I look to migrate this to Piston, which is free and open-source and has unlimited executions per day. For this, you need to go through the Piston docs and should send a post request to the piston server for code execution in the format they accept. The changes you would be required to make is in both the caucus and caucus-server repository, Changes to the backend Changes to the frontend |
@Rishabh-malhotraa Never mind! I signed up for a free account and the endpoints are working for me now |
@Rishabh-malhotraa here's the PR: Rishabh-malhotraa/caucus-server#6 I tested it locally, and it seemed to work fine without any modifications to the frontend. I'm no too familiar with this repo though, so let me know if I'm missing something! |
Jdoodle you are capped to 200 request per day, while piston is free and open source you can have your own docker container running or just use the public API for execution and the only limit is not more that 5 request per second 🚀.
There is no rush right now as we are not exceed more than 100 request per day, you can check the number of request left with the
http://caucus-app.herokuapp.com/api/credit-spent
https://github.com/engineer-man/piston
https://www.jdoodle.com/compiler-api
The text was updated successfully, but these errors were encountered: