aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorGravatar MarkKoz <[email protected]>2019-06-05 09:23:32 -0700
committerGravatar MarkKoz <[email protected]>2019-06-05 09:23:32 -0700
commit8ce01c95e4b1a31ac93c1fd7b9651a360b6ff3ed (patch)
tree4e46898f9a4124175558eeac7f898a9bfcd50420
parentAdd logging for NsJail (diff)
Revert "Add basic logger for responses"
Gunicorn's access logger makes the middleware redundant. This reverts commit c1c854e0ad4deb5c4fa8eb2a35b43a042f15255c.
-rw-r--r--snekbox/api/middleware/__init__.py3
-rw-r--r--snekbox/api/middleware/logger.py11
-rw-r--r--snekbox/api/snekapi.py3
3 files changed, 1 insertions, 16 deletions
diff --git a/snekbox/api/middleware/__init__.py b/snekbox/api/middleware/__init__.py
deleted file mode 100644
index ba97ca6..0000000
--- a/snekbox/api/middleware/__init__.py
+++ /dev/null
@@ -1,3 +0,0 @@
-from .logger import LoggingMiddleware
-
-__all__ = ("LoggingMiddleware",)
diff --git a/snekbox/api/middleware/logger.py b/snekbox/api/middleware/logger.py
deleted file mode 100644
index 393fc64..0000000
--- a/snekbox/api/middleware/logger.py
+++ /dev/null
@@ -1,11 +0,0 @@
-import logging
-
-log = logging.getLogger("snekbox.api")
-
-
-class LoggingMiddleware:
- """Log basic information about responses."""
-
- def process_response(self, req, resp, resource, req_succeeded):
- """Log the method, route, and status of a response."""
- log.info(f"{req.method} {req.relative_uri} {resp.status}")
diff --git a/snekbox/api/snekapi.py b/snekbox/api/snekapi.py
index 849e7d6..cb0356a 100644
--- a/snekbox/api/snekapi.py
+++ b/snekbox/api/snekapi.py
@@ -1,6 +1,5 @@
import falcon
-from .middleware import LoggingMiddleware
from .resources import EvalResource
@@ -22,6 +21,6 @@ class SnekAPI(falcon.API):
"""
def __init__(self, *args, **kwargs):
- super().__init__(middleware=[LoggingMiddleware()], *args, **kwargs)
+ super().__init__(*args, **kwargs)
self.add_route("/eval", EvalResource())