kallithea Changeset - 1a409593f352

Changeset - 1a409593f352

Parent rev.

Child rev.

[Not reviewed]

default

0 1 0

Mads Kiilerich - 6 years ago 2019-11-24 00:16:31
mads@kiilerich.com

Grafted from: 3028bfbcff88

app: drop finally handling in BaseVCSController.__call__

Our optional 'wrapper' middleware is logging response times more correctly
after b42ee1bdf082 - there is no point in logging the timing of the __call__ in
the main app.

Similarly, the session was removed too early. But TurboGears is already picking
up our db session and using DBSessionRemoverMiddleware to .remove() it ... and
at the right time. So just stop trying.

1 file changed with 0 insertions and 6 deletions:

kallithea/lib/base.py

0 comments (0 inline, 0 general)

kallithea/lib/base.py

➞

Show inline comments

@@ @@ -30,7 +30,6 @@ Original author and date, and relevant c @@
 import datetime
 import logging
 import time
 import traceback
 import warnings
@@ @@ -300,7 +299,6 @@ class BaseVCSController(object): @@
         return _get_ip_addr(environ)
     def __call__(self, environ, start_response):
         start = time.time()
         try:
             # try parsing a request for this VCS - if it fails, call the wrapped app
             parsed_request = self.parse_request(environ)
@@ @@ -343,10 +341,6 @@ class BaseVCSController(object): @@
         except webob.exc.HTTPException as e:
             return e(environ, start_response)
         finally:
             log_ = logging.getLogger('kallithea.' + self.__class__.__name__)
             log_.debug('Request time: %.3fs', time.time() - start)
             meta.Session.remove()
 class BaseController(TGController):

0 comments (0 inline, 0 general)