Issue Details (XML | Word | Printable)

Key: MODPYTHON-123
Type: New Feature New Feature
Status: Open Open
Priority: Major Major
Assignee: Unassigned
Reporter: Graham Dumpleton
Votes: 0
Watchers: 0
Operations

If you were logged in you would be able to see more operations.
mod_python

Provide handler hook point corresponding to ap_hook_map_to_storage().

Created: 08/Feb/06 08:02 AM   Updated: 01/Apr/06 02:16 PM
Return to search
Component/s: core
Affects Version/s: 3.3.x
Fix Version/s: None

Time Tracking:
Not Specified

Issue Links:
Blocker
 


 Description  « Hide
In between Apache phases:

    /* [2] filename-to-URI translation */
    ap_hook_translate_name(PythonTransHandler,
                           NULL, NULL, APR_HOOK_MIDDLE);

    /* [3] header parser */
    ap_hook_header_parser(PythonHeaderParserHandler,
                          NULL, NULL, APR_HOOK_MIDDLE);

There is actually another phase for mapping the file name against actual storage. The core (default) handler in Apache for this phase does various directory/file tree walking through filesystem directories assocated with Directory/File directives, looking at .htaccess files, evaluating whether symlinks can be followed etc. Basically it works out whether a file might exist and whether it is a candidate for use prior to applying later access/authentication phase tests.

In some cases you might want to stop all these checks happening, for example, in mod_proxy it stops it as the resources are actually on a different system and it would be a waste of time. Similarly, looks like Tomcat may also override it, possibly mapping the filename to its own concept of a file store.

In order to fill out the holes, seems it may be appropriate for mod_python to allow this phase to be replaced. Ie., have:

    /* [2 1/2] map to storage */
    ap_hook_map_to_storage(PythonStorageHandler,
                          NULL, NULL, APR_HOOK_MIDDLE);

Other changes in src/mod_python.c would need to be made as appropriate.


 All   Comments   Work Log   Change History   Subversion Commits      Sort Order: Ascending order - Click to sort in descending order
No work has yet been logged on this issue.