[Python-modules-commits] [twisted] 03/05: combinedlog

[Free Ekanayaka]

This is an automated email from the git hooks/post-receive script.

freee pushed a commit to branch patch-queue/master
in repository twisted.

commit ea8571ec989a617164a59453fc4adf272e7ab254
Author: Matthias Klose <doko at debian.org>
Date:   Thu Oct 20 04:34:06 2016 +0000

    combinedlog
    
     https://twistedmatrix.com/trac/ticket/7730
    
     for more details.
    Author: Free Ekanayaka <freee at debian.org>
    Bug-Debian: https://bugs.debian.org/772629
    Forwarded: yes
    Last-Update: 2014-11-09
    
    Gbp-Pq: Name combinedlog.patch
---
 twisted/web/http.py          |  2 +-
 twisted/web/test/test_web.py | 16 ++++++++--------
 2 files changed, 9 insertions(+), 9 deletions(-)

diff --git a/twisted/web/http.py b/twisted/web/http.py
index c7caba3..148fb86 100644
--- a/twisted/web/http.py
+++ b/twisted/web/http.py
@@ -2143,7 +2143,7 @@ def combinedLogFormatter(timestamp, request):
     referrer = _escape(request.getHeader(b"referer") or b"-")
     agent = _escape(request.getHeader(b"user-agent") or b"-")
     line = (
-        u'"%(ip)s" - - %(timestamp)s "%(method)s %(uri)s %(protocol)s" '
+        u'%(ip)s - - %(timestamp)s "%(method)s %(uri)s %(protocol)s" '
         u'%(code)d %(length)s "%(referrer)s" "%(agent)s"' % dict(
             ip=_escape(request.getClientIP() or b"-"),
             timestamp=timestamp,
diff --git a/twisted/web/test/test_web.py b/twisted/web/test/test_web.py
index 6f10648..f2cb628 100644
--- a/twisted/web/test/test_web.py
+++ b/twisted/web/test/test_web.py
@@ -1185,7 +1185,7 @@ class AccessLogTestsMixin(object):
 
         self.assertEqual(
             # Client IP
-            b'"1.2.3.4" '
+            b'1.2.3.4 '
             # Some blanks we never fill in
             b'- - '
             # The current time (circa 1234567890)
@@ -1285,7 +1285,7 @@ class CombinedLogFormatterTests(unittest.TestCase):
 
         line = http.combinedLogFormatter(timestamp, request)
         self.assertEqual(
-            u'"evil x-forwarded-for \\x80" - - [13/Feb/2009:23:31:30 +0000] '
+            u'evil x-forwarded-for \\x80 - - [13/Feb/2009:23:31:30 +0000] '
             u'"POS\\x81 /dummy HTTP/1.0" 123 - "evil \\x83" "evil \\x84"',
             line)
 
@@ -1377,7 +1377,7 @@ class LogEscapingTests(unittest.TestCase):
         self.site._logDateTime = "[%02d/%3s/%4d:%02d:%02d:%02d +0000]" % (
             25, 'Oct', 2004, 12, 31, 59)
         self.assertLogs(
-            b'"1.2.3.4" - - [25/Oct/2004:12:31:59 +0000] '
+            b'1.2.3.4 - - [25/Oct/2004:12:31:59 +0000] '
             b'"GET /dummy HTTP/1.0" 123 - "-" "-"\n')
 
 
@@ -1389,7 +1389,7 @@ class LogEscapingTests(unittest.TestCase):
             25, 'Oct', 2004, 12, 31, 59)
         self.request.method = b'G"T'
         self.assertLogs(
-            b'"1.2.3.4" - - [25/Oct/2004:12:31:59 +0000] '
+            b'1.2.3.4 - - [25/Oct/2004:12:31:59 +0000] '
             b'"G\\"T /dummy HTTP/1.0" 123 - "-" "-"\n')
 
 
@@ -1401,7 +1401,7 @@ class LogEscapingTests(unittest.TestCase):
             25, 'Oct', 2004, 12, 31, 59)
         self.request.uri = b'/dummy"withquote'
         self.assertLogs(
-            b'"1.2.3.4" - - [25/Oct/2004:12:31:59 +0000] '
+            b'1.2.3.4 - - [25/Oct/2004:12:31:59 +0000] '
             b'"GET /dummy\\"withquote HTTP/1.0" 123 - "-" "-"\n')
 
 
@@ -1413,7 +1413,7 @@ class LogEscapingTests(unittest.TestCase):
             25, 'Oct', 2004, 12, 31, 59)
         self.request.clientproto = b'HT"P/1.0'
         self.assertLogs(
-            b'"1.2.3.4" - - [25/Oct/2004:12:31:59 +0000] '
+            b'1.2.3.4 - - [25/Oct/2004:12:31:59 +0000] '
             b'"GET /dummy HT\\"P/1.0" 123 - "-" "-"\n')
 
 
@@ -1428,7 +1428,7 @@ class LogEscapingTests(unittest.TestCase):
             b'referer',
             b'http://malicious" ".website.invalid')
         self.assertLogs(
-            b'"1.2.3.4" - - [25/Oct/2004:12:31:59 +0000] '
+            b'1.2.3.4 - - [25/Oct/2004:12:31:59 +0000] '
             b'"GET /dummy HTTP/1.0" 123 - '
             b'"http://malicious\\" \\".website.invalid" "-"\n')
 
@@ -1443,7 +1443,7 @@ class LogEscapingTests(unittest.TestCase):
         self.request.requestHeaders.addRawHeader(b'user-agent',
                                                  b'Malicious Web" Evil')
         self.assertLogs(
-            b'"1.2.3.4" - - [25/Oct/2004:12:31:59 +0000] '
+            b'1.2.3.4 - - [25/Oct/2004:12:31:59 +0000] '
             b'"GET /dummy HTTP/1.0" 123 - "-" "Malicious Web\\" Evil"\n')
 
 

-- 
Alioth's /usr/local/bin/git-commit-notice on /srv/git.debian.org/git/python-modules/packages/twisted.git