durusmail: quixote-users: Patch to mod_python_handler.py needs testing
Patch to mod_python_handler.py needs testing
2001-11-13
2001-11-13
2001-11-13
2001-11-13
Patch to mod_python_handler.py needs testing
Greg Ward
2001-11-13
On 13 November 2001, I said:
> last week, Neil made this patch to Quixote's mod_python_handler.py:

Oops, it looks as though that patch depends on this patch:

--- publish.py  2001/11/02 18:01:20     1.102
+++ publish.py  2001/11/08 18:15:39     1.103
@@ -5,3 +5,3 @@

-__revision__ = "$Id: publish.py,v 1.102 2001/11/02 18:01:20 gward Exp $"
+__revision__ = "$Id: publish.py,v 1.103 2001/11/08 18:15:39 nascheme Exp $"

@@ -371,3 +371,3 @@
             request.response.set_header("Content-Type", "text/html")
-            admin = os.environ.get('SERVER_ADMIN',
+            admin = request.environ.get('SERVER_ADMIN',
                                    "email address unknown")
@@ -569,4 +569,4 @@
         script_name = "/".join([self.config.url_prefix] + path_components)
-        os.environ['SCRIPT_NAME'] = script_name
-        os.environ['PATH_INFO'] = '' # we always traverse the entire path
+        request.environ['SCRIPT_NAME'] = script_name
+        request.environ['PATH_INFO'] = '' # we always traverse the entire path

@@ -636,4 +636,4 @@

-    def publish (self, stdin, stdout, stderr):
-        """publish(stdin : file, stdout : file, stderr : file)
+    def publish (self, stdin, stdout, stderr, env):
+        """publish(stdin : file, stdout : file, stderr : file, env : dict)

@@ -643,6 +643,6 @@

-        request = self.parse_request(stdin, os.environ)
-
+        request = self.parse_request(stdin, env)
+
         try:
-            output = self.try_publish(request, os.environ.get('PATH_INFO', ''))
+            output = self.try_publish(request, env.get('PATH_INFO', ''))

@@ -676,4 +676,3 @@
             f = fcgi.FCGI()
-            os.environ = f.env
-            self.publish(f.inp, f.out, f.err)
+            self.publish(f.inp, f.out, f.err, f.env)
             f.Finish()

Please apply them *both* and let us know how it goes.  Thanks.

        Greg
--
Greg Ward - software developer                gward@mems-exchange.org
MEMS Exchange                            http://www.mems-exchange.org


reply