Ticket #17358: update_logging_style_patch.diff

File update_logging_style_patch.diff, 9.2 KB (added by Steve Pulec, 13 years ago)

  • django/core/handlers/base.py 

    diff --git a/django/core/handlers/base.py b/django/core/handlers/base.py
index 3606e18..a0918bf 100644
    a b class BaseHandler(object):  
    136136                    response = response.render()
    137137
    138138            except http.Http404, e:
    139                 logger.warning('Not Found: %s' % request.path,
     139                logger.warning('Not Found: %s', request.path,
    140140                            extra={
    141141                                'status_code': 404,
    142142                                'request': request
    … … class BaseHandler(object):  
    155155                            signals.got_request_exception.send(sender=self.__class__, request=request)
    156156            except exceptions.PermissionDenied:
    157157                logger.warning(
    158                     'Forbidden (Permission denied): %s' % request.path,
     158                    'Forbidden (Permission denied): %s', request.path,
    159159                    extra={
    160160                        'status_code': 403,
    161161                        'request': request
    … … class BaseHandler(object):  
    208208        if settings.DEBUG_PROPAGATE_EXCEPTIONS:
    209209            raise
    210210
    211         logger.error('Internal Server Error: %s' % request.path,
     211        logger.error('Internal Server Error: %s', request.path,
    212212            exc_info=exc_info,
    213213            extra={
    214214                'status_code': 500,

  • django/db/backends/util.py 

    diff --git a/django/db/backends/util.py b/django/db/backends/util.py
index b0463b7..9706268 100644
    a b class CursorDebugWrapper(CursorWrapper):  
    4242                'sql': sql,
    4343                'time': "%.3f" % duration,
    4444            })
    45             logger.debug('(%.3f) %s; args=%s' % (duration, sql, params),
     45            logger.debug('(%.3f) %s; args=%s', duration, sql, params,
    4646                extra={'duration':duration, 'sql':sql, 'params':params}
    4747            )
    4848
    … … class CursorDebugWrapper(CursorWrapper):  
    5757                'sql': '%s times: %s' % (len(param_list), sql),
    5858                'time': "%.3f" % duration,
    5959            })
    60             logger.debug('(%.3f) %s; args=%s' % (duration, sql, param_list),
     60            logger.debug('(%.3f) %s; args=%s', duration, sql, param_list,
    6161                extra={'duration':duration, 'sql':sql, 'params':param_list}
    6262            )
    6363

  • django/middleware/common.py 

    diff --git a/django/middleware/common.py b/django/middleware/common.py
index 4dd2e14..aaa094d 100644
    a b class CommonMiddleware(object):  
    4242        if 'HTTP_USER_AGENT' in request.META:
    4343            for user_agent_regex in settings.DISALLOWED_USER_AGENTS:
    4444                if user_agent_regex.search(request.META['HTTP_USER_AGENT']):
    45                     logger.warning('Forbidden (User agent): %s' % request.path,
     45                    logger.warning('Forbidden (User agent): %s', request.path,
    4646                        extra={
    4747                            'status_code': 403,
    4848                            'request': request

  • django/middleware/csrf.py 

    diff --git a/django/middleware/csrf.py b/django/middleware/csrf.py
index 37f92b1..856c266 100644
    a b class CsrfViewMiddleware(object):  
    134134                # we can use strict Referer checking.
    135135                referer = request.META.get('HTTP_REFERER')
    136136                if referer is None:
    137                     logger.warning('Forbidden (%s): %s' % (REASON_NO_REFERER, request.path),
     137                    logger.warning('Forbidden (%s): %s', REASON_NO_REFERER, request.path,
    138138                        extra={
    139139                            'status_code': 403,
    140140                            'request': request,
    … … class CsrfViewMiddleware(object):  
    146146                good_referer = 'https://%s/' % request.get_host()
    147147                if not same_origin(referer, good_referer):
    148148                    reason = REASON_BAD_REFERER % (referer, good_referer)
    149                     logger.warning('Forbidden (%s): %s' % (reason, request.path),
     149                    logger.warning('Forbidden (%s): %s', reason, request.path,
    150150                        extra={
    151151                            'status_code': 403,
    152152                            'request': request,
    … … class CsrfViewMiddleware(object):  
    158158                # No CSRF cookie. For POST requests, we insist on a CSRF cookie,
    159159                # and in this way we can avoid all CSRF attacks, including login
    160160                # CSRF.
    161                 logger.warning('Forbidden (%s): %s' % (REASON_NO_CSRF_COOKIE, request.path),
     161                logger.warning('Forbidden (%s): %s', REASON_NO_CSRF_COOKIE, request.path,
    162162                    extra={
    163163                        'status_code': 403,
    164164                        'request': request,
    … … class CsrfViewMiddleware(object):  
    177177                request_csrf_token = request.META.get('HTTP_X_CSRFTOKEN', '')
    178178
    179179            if not constant_time_compare(request_csrf_token, csrf_token):
    180                 logger.warning('Forbidden (%s): %s' % (REASON_BAD_TOKEN, request.path),
     180                logger.warning('Forbidden (%s): %s', REASON_BAD_TOKEN, request.path,
    181181                    extra={
    182182                        'status_code': 403,
    183183                        'request': request,

  • django/views/decorators/http.py 

    diff --git a/django/views/decorators/http.py b/django/views/decorators/http.py
index 495a7b1..bf96371 100644
    a b def require_http_methods(request_method_list):  
    3131        @wraps(func, assigned=available_attrs(func))
    3232        def inner(request, *args, **kwargs):
    3333            if request.method not in request_method_list:
    34                 logger.warning('Method Not Allowed (%s): %s' % (request.method, request.path),
     34                logger.warning('Method Not Allowed (%s): %s', request.method, request.path,
    3535                    extra={
    3636                        'status_code': 405,
    3737                        'request': request
    … … def condition(etag_func=None, last_modified_func=None):  
    121121                    if request.method in ("GET", "HEAD"):
    122122                        response = HttpResponseNotModified()
    123123                    else:
    124                         logger.warning('Precondition Failed: %s' % request.path,
     124                        logger.warning('Precondition Failed: %s', request.path,
    125125                            extra={
    126126                                'status_code': 412,
    127127                                'request': request
    … … def condition(etag_func=None, last_modified_func=None):  
    130130                        response = HttpResponse(status=412)
    131131                elif if_match and ((not res_etag and "*" in etags) or
    132132                        (res_etag and res_etag not in etags)):
    133                     logger.warning('Precondition Failed: %s' % request.path,
     133                    logger.warning('Precondition Failed: %s', request.path,
    134134                        extra={
    135135                            'status_code': 412,
    136136                            'request': request

  • django/views/generic/base.py 

    diff --git a/django/views/generic/base.py b/django/views/generic/base.py
index ea14281..556d0eb 100644
    a b class View(object):  
    6868
    6969    def http_method_not_allowed(self, request, *args, **kwargs):
    7070        allowed_methods = [m for m in self.http_method_names if hasattr(self, m)]
    71         logger.warning('Method Not Allowed (%s): %s' % (request.method, request.path),
     71        logger.warning('Method Not Allowed (%s): %s', request.method, request.path,
    7272            extra={
    7373                'status_code': 405,
    7474                'request': self.request
    … … class RedirectView(View):  
    157157            else:
    158158                return http.HttpResponseRedirect(url)
    159159        else:
    160             logger.warning('Gone: %s' % self.request.path,
     160            logger.warning('Gone: %s', self.request.path,
    161161                        extra={
    162162                            'status_code': 410,
    163163                            'request': self.request

  • django/views/generic/simple.py 

    diff --git a/django/views/generic/simple.py b/django/views/generic/simple.py
index c3cf407..110932a 100644
    a b def redirect_to(request, url, permanent=True, query_string=False, **kwargs):  
    6060        klass = permanent and HttpResponsePermanentRedirect or HttpResponseRedirect
    6161        return klass(url)
    6262    else:
    63         logger.warning('Gone: %s' % request.path,
     63        logger.warning('Gone: %s', request.path,
    6464                    extra={
    6565                        'status_code': 410,
    6666                        'request': request

  • tests/regressiontests/views/views.py 

    diff --git a/tests/regressiontests/views/views.py b/tests/regressiontests/views/views.py
index f2b4e79..1d17e6e 100644
    a b def send_log(request, exc_info):  
    149149        ][0]
    150150    orig_filters = admin_email_handler.filters
    151151    admin_email_handler.filters = []
    152     logger.error('Internal Server Error: %s' % request.path,
     152    logger.error('Internal Server Error: %s', request.path,
    153153        exc_info=exc_info,
    154154        extra={
    155155            'status_code': 500,
Back to Top