data-8 / Gofer-Grader

Small autograding library
http://okgrade.readthedocs.io/en/latest/
BSD 3-Clause "New" or "Revised" License
16 stars 19 forks source link

Add server requirements and use async post #14

Closed rameshvs closed 5 years ago

rameshvs commented 5 years ago

This PR has three changes:

  1. The post method now uses async def, since the tornado.web.asynchronous decorator is deprecated.
  2. Uses NamedTemporaryFile to create a temp file rather than manually looping (and potentially being susceptible to a race condition and/or security threat, as unlikely as those may be).
  3. I added a draft of a requirements.txt file for the server.

Disclaimer: I was not able to test this with the actual grading script since I don't yet have access to it.