[Zope-Checkins] CVS: Zope/ZServer - FCGIServer.py:1.16 PCGIServer.py:1.22

[Matt Behrens]

Update of /cvs-repository/Zope/ZServer
In directory cvs.zope.org:/tmp/cvs-serv16915/ZServer

Modified Files:
	FCGIServer.py PCGIServer.py 
Log Message:
#185 plus fix ajung's fix for #341: repair Z2.log for [FP]CGIServer, and
give them extended log format like HTTPServer


=== Zope/ZServer/FCGIServer.py 1.15 => 1.16 ===
         DebugLogger.log('E', id(self))
 
-        user_agent=self.get_header('user-agent')
-        if not user_agent: user_agent=''
-        referer=self.get_header('referer')
-        if not referer: referer=''  
+        if self.env.has_key('HTTP_USER_AGENT'):
+            user_agent=self.env['HTTP_USER_AGENT']
+        else:
+            user_agent=''
+        if self.env.has_key('HTTP_REFERER'):
+            referer=self.env['HTTP_REFERER']
+        else:
+            referer=''
        
         if self.env.has_key('PATH_INFO'):
             path=self.env['PATH_INFO']
@@ -453,7 +457,7 @@
         if self.addr:
             self.server.logger.log (
                 self.addr[0],
-                '%s - - [%s] "%s %s" %d %d' % (
+                '%s - - [%s] "%s %s" %d %d "%s" "%s"' % (
                     self.addr[1],
                     time.strftime (
                     '%d/%b/%Y:%H:%M:%S ',
@@ -466,7 +470,7 @@
         else:
             self.server.logger.log (
                 '127.0.0.1 ',
-                '- - [%s] "%s %s" %d %d' % (
+                '- - [%s] "%s %s" %d %d "%s" "%s"' % (
                     time.strftime (
                     '%d/%b/%Y:%H:%M:%S ',
                     time.localtime(time.time())


=== Zope/ZServer/PCGIServer.py 1.21 => 1.22 ===
      
     def log_request(self, bytes):
+        if self.env.has_key('HTTP_USER_AGENT'):
+            user_agent=self.env['HTTP_USER_AGENT']
+        else:
+            user_agent=''
+        if self.env.has_key('HTTP_REFERER'):
+            referer=self.env['HTTP_REFERER']
+        else:
+            referer=''
+
         if self.env.has_key('PATH_INFO'):
             path=self.env['PATH_INFO']
         else:
@@ -153,24 +162,26 @@
         if addr and type(addr) is TupleType:
             self.server.logger.log (
                 addr[0],
-                '%d - - [%s] "%s %s" %d %d' % (
+                '%d - - [%s] "%s %s" %d %d "%s" "%s"' % (
                     addr[1],
                     time.strftime (
                     '%d/%b/%Y:%H:%M:%S ',
-                    time.gmtime(time.time())
+                    time.localtime(time.time())
                     ) + tz_for_log,
-                    method, path, self.reply_code, bytes
+                    method, path, self.reply_code, bytes,
+                    referer, user_agent
                     )
                 )
         else:
             self.server.logger.log (
                 '127.0.0.1',
-                '- - [%s] "%s %s" %d %d' % (
+                ' - - [%s] "%s %s" %d %d "%s" "%s"' % (
                     time.strftime (
                     '%d/%b/%Y:%H:%M:%S ',
                     time.gmtime(time.time())
                     ) + tz_for_log,
-                    method, path, self.reply_code, bytes
+                    method, path, self.reply_code, bytes,
+                    referer, user_agent
                     )
                 )