Log start and end of requests; some cleanup.

lakesuperior/endpoints/ldp.py

@@ -47,17 +47,6 @@ accept_rdf = (
     'text/rdf+n3',
     'text/turtle',
 )
-#allow = (
-#    'COPY',
-#    'DELETE',
-#    'GET',
-#    'HEAD',
-#    'MOVE',
-#    'OPTIONS',
-#    'PATCH',
-#    'POST',
-#    'PUT',
-#)
 
 std_headers = {
     'Accept-Patch' : ','.join(accept_patch),
@@ -77,6 +66,11 @@ def bp_url_value_preprocessor(endpoint, values):
     g.webroot = request.host_url + g.url_prefix
 
 
+@ldp.before_request
+def log_request_start():
+    logger.info('\n\n** Start {} {} **'.format(request.method, request.url))
+
+
 @ldp.before_request
 def instantiate_toolbox():
     g.tbox = Toolbox()
@@ -88,6 +82,13 @@ def request_timestamp():
     g.timestamp_term = Literal(g.timestamp, datatype=XSD.dateTime)
 
 
+@ldp.after_request
+def log_request_end(rsp):
+    logger.info('** End {} {} **\n\n'.format(request.method, request.url))
+
+    return rsp
+
+
 ## REST SERVICES ##
 
 @ldp.route('/<path:uuid>', methods=['GET'], strict_slashes=False)
@@ -131,6 +132,7 @@ def get_resource(uuid, force_rdf=False):
         else:
             return send_file(rsrc.local_path, as_attachment=True,
                     attachment_filename=rsrc.filename)
+            logger.info('Streaming out binary content.')
 
 
 @ldp.route('/<path:parent>', methods=['POST'], strict_slashes=False)

lakesuperior/toolbox.py

@@ -4,8 +4,8 @@ import pickle
 from collections import defaultdict
 from hashlib import sha1
 
-from flask import request, g
-from rdflib.term import Literal, URIRef, Variable
+from flask import g
+from rdflib.term import URIRef, Variable
 
 from lakesuperior.dictionaries.namespaces import ns_collection as nsc
 

tests/endpoints/test_ldp.py

@@ -3,14 +3,14 @@ import uuid
 
 from hashlib import sha1
 
-from flask import url_for, g
+from flask import g
 from rdflib import Graph
 from rdflib.namespace import RDF
 from rdflib.term import Literal, URIRef
 
 from lakesuperior.dictionaries.namespaces import ns_collection as nsc
 from lakesuperior.model.ldpr import Ldpr
-from lakesuperior.toolbox import Toolbox
+
 
 @pytest.fixture(scope='module')
 def random_uuid():