search

amplab / ampcrowd

A RESTful web service that runs microtasks across multiple crowds, provides quality control techniques, and is easily extensible.
https://amplab.github.io/ampcrowd
Apache License 2.0
51 stars 18 forks source link

Better logging + small improvements #5

Closed EntilZha closed 9 years ago

EntilZha commented 9 years ago

Most of changes here are to add better logging and error handling.

  1. Defined logging block for both dev and prod. prod will log as it did before, although I would like to change this if docker is detected in some way. Not a big priority though. In dev, errors are logged to standard out/error which has been helpful (especially with setting PYTHONUNBUFFERED=1 in the Dockerfile (this change is already in).
  2. Defined error handling middleware/exception for AMT related errors. This should better warn users and inform them the source of the error (in my case not using https).
  3. Improved error handling on post.py as well.
thisisdhaas commented 9 years ago

This looks awesome. Handle the one comment, and I'll merge away.