+1 on this for me.
If there are no objections I'll apply the patch and commit to svn.

Regards,
Jim

Graham Dumpleton (JIRA) wrote:
     [ http://issues.apache.org/jira/browse/MODPYTHON-67?page=all ]

Graham Dumpleton updated MODPYTHON-67:
--------------------------------------

    Attachment: requestobject.c.diff

Patch for requestobject.c  which makes req.path_info writable.


Make req.path_info writable.
----------------------------

        Key: MODPYTHON-67
        URL: http://issues.apache.org/jira/browse/MODPYTHON-67
    Project: mod_python
       Type: New Feature
 Components: core
   Versions: 3.2.0
   Reporter: Graham Dumpleton
Attachments: requestobject.c.diff

In current version of mod_python req.filename is writable, but
req.path_info isn't.
If req.path_info were writable it would more easily allow a middleware
stack to be implemented in mod_python. This is because a middleware
component could modify req.path_info as necessary as the path info is
progressively processed. This would be better than a middleware
component having to create a parallel request attribute just for this
purpose as by being able to change req.path_info existing handlers such
as mod_python.publisher which already look at req.filename and
req.path_info could still be utilised in various ways within the
middleware stack. Same for any handlers which third parties may have
developed. Thus would eliminate the need to copy existing handlers and
modify them to get them to work, a middleware component would simply
need to setup req.filename and req.path_info as appropriate and the
existing handlers would work unmodified.



Reply via email to