Changeset 391 for python/trunk/Lib/wsgiref

Timestamp:

Mar 19, 2014, 11:31:01 PM (11 years ago)

Author:

dmik

Message:

python: Merge vendor 2.7.6 to trunk.

Location:

python/trunk

Files:

• . (modified) (1 prop)
• Lib/wsgiref/handlers.py (modified) (10 diffs)
• Lib/wsgiref/headers.py (modified) (5 diffs)
• Lib/wsgiref/simple_server.py (modified) (4 diffs)
• Lib/wsgiref/util.py (modified) (4 diffs)
• Lib/wsgiref/validate.py (modified) (2 diffs)

python/trunk/Lib/wsgiref/handlers.py

@@ -40 +40 @@
 
 
-
 class BaseHandler:
     """Manage the invocation of a WSGI application"""
@@ -65 +64 @@
     # Error handling (also per-subclass or per-instance)
     traceback_limit = None  # Print entire traceback to self.get_stderr()
-    error_status = "500 Dude, this is whack!"
+    error_status = "500 Internal Server Error"
     error_headers = [('Content-Type','text/plain')]
     error_body = "A server error occurred.  Please contact the administrator."
@@ -74 +73 @@
     headers = None
     bytes_sent = 0
-
-
-
-
-
-
-
 
     def run(self, application):
@@ -131 +123 @@
         'self.close()' once the response is finished.
         """
-        if not self.result_is_file() or not self.sendfile():
-            for data in self.result:
-                self.write(data)
-            self.finish_content()
-        self.close()
+        try:
+            if not self.result_is_file() or not self.sendfile():
+                for data in self.result:
+                    self.write(data)
+                self.finish_content()
+        finally:
+            self.close()
 
 
@@ -161 +155 @@
         Subclasses can extend this to add other defaults.
         """
-        if not self.headers.has_key('Content-Length'):
+        if 'Content-Length' not in self.headers:
             self.set_content_length()
 
@@ -196 +190 @@
             if self.client_is_modern():
                 self._write('HTTP/%s %s\r\n' % (self.http_version,self.status))
-                if not self.headers.has_key('Date'):
+                if 'Date' not in self.headers:
                     self._write(
                         'Date: %s\r\n' % format_date_time(time.time())
                     )
-                if self.server_software and not self.headers.has_key('Server'):
+                if self.server_software and 'Server' not in self.headers:
                     self._write('Server: %s\r\n' % self.server_software)
         else:
@@ -249 +243 @@
         """Ensure headers and content have both been sent"""
         if not self.headers_sent:
-            self.headers['Content-Length'] = "0"
+            # Only zero Content-Length if not set by the application (so
+            # that HEAD requests can be satisfied properly, see #3839)
+            self.headers.setdefault('Content-Length', "0")
             self.send_headers()
         else:
@@ -360 +356 @@
 
 
-
-
-
-
-
-
-
-
-
 class SimpleHandler(BaseHandler):
     """Handler that's just initialized with streams, environment, etc.
@@ -434 +421 @@
 
 
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
 class CGIHandler(BaseCGIHandler):
 
@@ -479 +449 @@
             multithread=False, multiprocess=True
         )
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-#

python/trunk/Lib/wsgiref/headers.py

@@ -26 +26 @@
     else:
         return param
-
-
-
-
-
-
-
-
-
-
-
-
 
 
@@ -77 +65 @@
         return self.get(name)
 
-
-
-
-
     def has_key(self, name):
         """Return true if the message contains the header."""
@@ -118 +102 @@
         """
         return [k for k, v in self._headers]
-
-
-
 
     def values(self):
@@ -162 +143 @@
             return result
 
-
     def add_header(self, _name, _value, **_params):
         """Extended header setting.
@@ -188 +168 @@
                 parts.append(_formatparam(k.replace('_', '-'), v))
         self._headers.append((_name, "; ".join(parts)))
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-#

python/trunk/Lib/wsgiref/simple_server.py

@@ -38 +38 @@
 
 
-
-
 class WSGIServer(HTTPServer):
 
@@ -66 +64 @@
     def set_app(self,application):
         self.application = application
-
-
-
-
-
-
-
-
-
-
-
-
 
 
@@ -140 +126 @@
 
 
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
 def demo_app(environ,start_response):
     from StringIO import StringIO
@@ -191 +154 @@
     webbrowser.open('http://localhost:8000/xyz?abc')
     httpd.handle_request()  # serve one request, then exit
-
-
-
-
-
-
-
-
-
-
-
-
-#
+    httpd.server_close()

python/trunk/Lib/wsgiref/util.py

@@ -33 +33 @@
         raise StopIteration
 
-
-
-
-
-
-
-
 def guess_scheme(environ):
     """Return a guess for whether 'wsgi.url_scheme' should be 'http' or 'https'
@@ -72 +65 @@
     url = application_uri(environ)
     from urllib import quote
-    path_info = quote(environ.get('PATH_INFO',''))
+    path_info = quote(environ.get('PATH_INFO',''),safe='/;=,')
     if not environ.get('SCRIPT_NAME'):
         url += path_info[1:]
@@ -162 +155 @@
 
 
-
 _hoppish = {
     'connection':1, 'keep-alive':1, 'proxy-authenticate':1,
@@ -172 +164 @@
     """Return true if 'header_name' is an HTTP/1.1 "Hop-by-Hop" header"""
     return _hoppish(header_name.lower())
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-#

python/trunk/Lib/wsgiref/validate.py

@@ -135 +135 @@
     middleware will check for WSGI compliancy on a number of levels.
     This middleware does not modify the request or response in any
-    way, but will throw an AssertionError if anything seems off
+    way, but will raise an AssertionError if anything seems off
     (except for a failure to close the application iterator, which
-    will be printed to stderr -- there's no way to throw an exception
+    will be printed to stderr -- there's no way to raise an exception
     at that point).
     """
@@ -346 +346 @@
 
     if not environ.get('SCRIPT_NAME'):
-        assert_(environ.has_key('PATH_INFO'),
+        assert_('PATH_INFO' in environ,
             "One of SCRIPT_NAME or PATH_INFO are required (PATH_INFO "
             "should at least be '/' if SCRIPT_NAME is empty)")