Ticket #600: forward.patch

_cpconfig.py

@@ -48 +48 @@
-        _cp_config = {'tools.gzip.on': True}
-        
-        def index(self):
-
+
-        index.exposed = True
-recursive_redirect': Tru
+show_tracebacks': Fals
-
-
-Namespaces

_cpdispatch.py

@@ -308 +308 @@
-                merge(app.config[curpath])
-        
-        return handler
+
+
+def XMLRPCDispatcher(next_dispatcher=Dispatcher()):
+    from cherrypy.lib import xmlrpc
+    def xmlrpc_dispatch(path_info):
+        path_info = xmlrpc.patched_path(path_info)
+        return next_dispatcher(path_info)
+    return xmlrpc_dispatch
+
+
+def VirtualHost(next_dispatcher=Dispatcher(), use_x_forwarded_host=True, **domains):
+    """Select a different handler based on the Host header.
+    
+    Useful when running multiple sites within one CP server.
+    
+    From http://groups.google.com/group/cherrypy-users/browse_thread/thread/f393540fe278e54d:
+    
+    For various reasons I need several domains to point to different parts of a
+    single website structure as well as to their own "homepage"   EG
+    
+    http://www.mydom1.com  ->  root
+    http://www.mydom2.com  ->  root/mydom2/
+    http://www.mydom3.com  ->  root/mydom3/
+    http://www.mydom4.com  ->  under construction page
+    
+    but also to have  http://www.mydom1.com/mydom2/  etc to be valid pages in
+    their own right.
+    """
+    from cherrypy.lib import http
+    def vhost_dispatch(path_info):
+        header = cherrypy.request.headers.get
+        
+        domain = header('Host', '')
+        if use_x_forwarded_host:
+            domain = header("X-Forwarded-Host", domain)
+        
+        prefix = domains.get(domain, "")
+        if prefix:
+            path_info = http.urljoin(prefix, path_info)
+        
+        return next_dispatcher(path_info)
+    return vhost_dispatch
+

_cperror.py

@@ -12 +12 @@
-class InternalRedirect(CherryPyException):
-    """Exception raised to switch to the handler for a different URL.
-    
-
-
-
-
+
-    """
-    
-    def __init__(self, path):

_cprequest.py

@@ -135 +135 @@
-class Request(object):
-    """An HTTP request."""
-    
+    prev = None
+    
-    # Conversation/connection attributes
-    local = http.Host("localhost", 80)
-    remote = http.Host("localhost", 1111)
@@ -276 +278 @@
-            # path_info should be the path from the
-            # app root (script_name) to the handler.
-            self.script_name = self.app.script_name
-
+i = p
-            
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
+
+
+
-            raise
-        except:
-            if self.throw_errors:
@@ -383 +365 @@
-            self.handle_error(sys.exc_info())
-    
-    def process_headers(self):
-        # Guard against re-reading body (e.g. on InternalRedirect)
-        if self.headers_read:
-            return
-        self.headers_read = True
-        
-        self.params = http.parse_query_string(self.query_string)
-        
-        # Process the headers into self.headers
@@ -451 +428 @@
-    
-    def process_body(self):
-        """Convert request.rfile into request.params (or request.body)."""
-        # Guard against re-reading body (e.g. on InternalRedirect)
-        if self.body_read:
-            return
-        self.body_read = True
-        
-        # FieldStorage only recognizes POST, so fake it.
-        methenv = {'REQUEST_METHOD': "POST"}
-        try:

_cptools.py

@@ -214 +214 @@
-    index = __call__
-
-
-class XMLRPCTool(object):
-    """Tool for using XMLRPC over HTTP.
-    
-    Python's None value cannot be used in standard XML-RPC; to allow
-    using it via an extension, provide a true value for allow_none.
-    """
-    
-    def _setup(self):
-        """Hook this tool into cherrypy.request."""
-        request = cherrypy.request
-        
-        # Guard against running this method twice.
-        if hasattr(request, 'xmlrpc'):
-            return
-        request.xmlrpc = True
-        
-        request.error_response = _xmlrpc.on_error
-        path_info = request.path_info
-        ppath = _xmlrpc.patched_path(path_info)
-        if ppath != path_info:
-            raise cherrypy.InternalRedirect(ppath)
-
-
-class WSGIAppTool(HandlerTool):
-    """A tool for running any WSGI middleware/application within CP.
-    
@@ -332 +309 @@
-_d.session_auth = SessionAuthTool(cptools.session_auth)
-_d.proxy = Tool('before_request_body', cptools.proxy, priority=30)
-_d.response_headers = Tool('on_start_resource', cptools.response_headers)
-_d.virtual_host = Tool('on_start_resource', cptools.virtual_host, priority=40)
-_d.log_tracebacks = Tool('before_error_response', cptools.log_traceback)
-_d.log_headers = Tool('before_error_response', cptools.log_request_headers)
-_d.err_redirect = ErrorTool(cptools.redirect)
@@ -345 +321 @@
-_d.staticfile = HandlerTool(static.staticfile)
-# _sessions.init must be bound after headers are read
-_d.sessions = SessionTool('before_request_body', _sessions.init)
-XMLRPCTool(
+ErrorTool(_xmlrpc.on_error
-_d.wsgiapp = WSGIAppTool(_wsgiapp.run)
-_d.caching = CachingTool('before_handler', _caching.get, 'caching')
-_d.expires = Tool('before_finalize', _caching.expires)

_cptree.py

@@ -1 +1 @@
-"""CherryPy Application and Tree objects."""
-
-import os
+import StringIO
-import sys
-import cherrypy
-from cherrypy import _cpconfig, _cplogging, tools
@@ -86 +87 @@
-        remote = http.Host(env('REMOTE_ADDR', ''),
-                           int(env('REMOTE_PORT', -1)),
-                           env('REMOTE_HOST', ''))
-
-
-
-
+
+
-        # LOGON_USER is served by IIS, and is the name of the
-        # user after having been mapped to a local account.
-        # Both IIS and Apache set REMOTE_USER, when possible.
-request.
+
-        
-
-
-
+
+
+
+
+
+
-        
-
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
-        
-        path = env('SCRIPT_NAME', '') + env('PATH_INFO', '')
-        response = request.run(environ['REQUEST_METHOD'], path,
-                               env('QUERY_STRING', ''),
-                               env('SERVER_PROTOCOL'),
-                               translate_headers(environ),
-                               environ['wsgi.input'])
-        s, h, b = response.status, response.header_list, response.body
-        exc = None
-    except (KeyboardInterrupt, SystemExit), ex:

lib/cptools.py

@@ -272 +272 @@
-                 for k in dir(SessionAuth) if not k.startswith("__")])
-
-
-def virtual_host(use_x_forwarded_host=True, **domains):
-    """Redirect internally based on the Host header.
-    
-    Useful when running multiple sites within one CP server.
-    
-    From http://groups.google.com/group/cherrypy-users/browse_thread/thread/f393540fe278e54d:
-    
-    For various reasons I need several domains to point to different parts of a
-    single website structure as well as to their own "homepage"   EG
-    
-    http://www.mydom1.com  ->  root
-    http://www.mydom2.com  ->  root/mydom2/
-    http://www.mydom3.com  ->  root/mydom3/
-    http://www.mydom4.com  ->  under construction page
-    
-    but also to have  http://www.mydom1.com/mydom2/  etc to be valid pages in
-    their own right.
-    """
-    request = cherrypy.request
-    
-    # Guard against running twice.
-    if hasattr(request, "virtual_prefix"):
-        return
-    
-    domain = request.headers.get('Host', '')
-    if use_x_forwarded_host:
-        domain = request.headers.get("X-Forwarded-Host", domain)
-    
-    request.virtual_prefix = prefix = domains.get(domain, "")
-    if prefix:
-        raise cherrypy.InternalRedirect(_http.urljoin(prefix, request.path_info))
-
-def log_traceback():
-    """Write the last error's traceback to the cherrypy error log."""
-    from cherrypy import _cperror

test/test_core.py

@@ -164 +164 @@
-            raise cherrypy.InternalRedirect("cousin?t=6")
-        
-        def cousin(self, t):
-
-
+
+
-        
-        def petshop(self, user_id):
-            if user_id == "parrot":
@@ -173 +173 @@
-                raise cherrypy.InternalRedirect('/image/getImagesByUser?user_id=slug')
-            elif user_id == "terrier":
-                # Trade it for a fish when redirecting
-
-
+
-            else:
-                # This should pass the user_id through to getImagesByUser
-'
+?user_id=%s' % user_id
-        
-        # We support Python 2.3, but the @-deco syntax would look like this:
-        # @tools.login_redir()
@@ -606 +605 @@
-        self.assertStatus(200)
-        
-        # Relative path in InternalRedirect.
-
+
-        self.getPage("/internalredirect/relative?a=3&b=5")
-
-
+
-        self.assertStatus(200)
-        
-        # InternalRedirect on error

test/test_virtualhost.py

@@ -3 +3 @@
-test.prefer_parent_path()
-
-import cherrypy
+from cherrypy import _cpdispatch
-
-def setup_server():
-    class Root:
@@ -38 +39 @@
-    root = Root()
-    root.mydom2 = VHost("Domain 2")
-    root.mydom3 = VHost("Domain 3")
-
+
+
+
+
+
+
+
-    
-
-
-
-
-
-
-
+
-
-from cherrypy.test import helper
-
@@ -76 +77 @@
-        self.getPage("/vmethod/pos", [('Host', 'www.mydom3.com')])
-        self.assertBody("You sent 'pos'")
-        
+        # Test that cherrypy.url uses the browser url, not the virtual url
-        self.getPage("/url", [('Host', 'www.mydom2.com')])
-        self.assertBody("http://www.mydom2.com/nextpage")
-

test/test_xmlrpc.py

@@ -2 +2 @@
-test.prefer_parent_path()
-import xmlrpclib
-
+from cherrypy import _cpdispatch
-
+
-def setup_server():
-    import cherrypy
-    from cherrypy import _cptools
@@ -57 +59 @@
-
-    root = Root()
-    root.xmlrpc = XmlRpc()
-
+
+
+
-    cherrypy.config.update({'environment': 'test_suite'})
-
-