<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Tue, Nov 25, 2014 at 4:26 PM, Free Ekanayaka <span dir="ltr"><<a href="mailto:free@64studio.com" target="_blank">free@64studio.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex"><div dir="ltr"><div>Hi,</div><div><br></div><div>it was</div><div><br></div><div>            line = '%s - - %s "%s" %d %s "%s" "%s"\n' % (</div><div><br></div><div>in 13.2.0 (twisted/web/http.py line 1920), and it's</div><div><br></div><div>        u'"%(ip)s" - - %(timestamp)s "%(method)s %(uri)s %(protocol)s" '</div><div><br></div><div>so basically the client IP now gets wrapped within double quotes, e.g. a</div><div>log line that was looking like:</div><div><br></div><div>1.2.3.4 - - [25/Oct/2004:12:31:59 +0000] "GET /dummy HTTP/1.0" 123 - "-" "-"</div><div><br></div><div>it now looks like:</div><div><br></div><div>"1.2.3.4" - - [25/Oct/2004:12:31:59 +0000] "GET /dummy HTTP/1.0" 123 - "-" "-"</div><div><br></div><div>as one can see in the unit tests in test_web.py too.</div><div><br></div><div>What's the reason for this change?</div><div><br></div><div>It feels it can potentially break code that parses log files and it also</div><div>seems to diverge from the format described on the Apache web site:</div><div><br></div><div><a href="http://httpd.apache.org/docs/1.3/logs.html" target="_blank">http://httpd.apache.org/docs/1.3/logs.html</a></div></div></blockquote><div><br></div><div>I've inspected a bit the diff that introduced this change and read the comments</div><div>on the relevant ticket, but couldn't figure out the reason for the change, so I</div><div>filed #7730 to keep track of the issue.</div><div><br></div></div></div></div>