ubc / iPeer

Peer Evaluation System
http://ipeer.ctlt.ubc.ca
Other
17 stars 18 forks source link

Change Canvas API POST requests to send data in body rather than URI #590

Closed wynnset closed 6 years ago

wynnset commented 6 years ago

We were having an issue where grades could not be sync'd for one of the courses that had a lot of groups. After investigating with live data, found out that the Canvas API server was returning a "URI Too Long" error. We seem to have implemented the request in a way where even POST data is sent as parameters in the URI. I've now changed this to instead send it in the body, like how a form is submitted. This should prevent this error in the future.