jean-frederic clere wrote:
Henri Gomez wrote:

jean-frederic clere wrote:


Mladen Turk wrote:


Hi,

I finally have a couple of days to spare on JK2.
Since Henri is doing great job on Apache side, I'm willing to clean
the IIS
port.

Are there any pending issues that I can help with, so we can make the
release.



Bugzilla reports 59 bugs... Help is _needed_ !!!


Mladen, JF, Guenter, what about the APR_HOOK_MIDDLE in jk2_translate ?


I am +0 for an APR_HOOK_MIDDLE, but I think we have to make sure mod_rewrite is
before us:
+++
 static const char * const aszPre[] = { "mod_rewrite.c", NULL };

 ap_hook_translate_name(jk2_translate,aszPre,NULL,APR_HOOK_MIDDLE);
+++
I have no problems with the APR_HOOK_FIRST and mod_dav.

I see that comment in mod_file_cache :


static void register_hooks(apr_pool_t *p)
{
ap_hook_handler(file_cache_handler, NULL, NULL, APR_HOOK_LAST);
ap_hook_post_config(file_cache_post_config, NULL, NULL, APR_HOOK_MIDDLE);
ap_hook_translate_name(file_cache_xlat, NULL, NULL, APR_HOOK_MIDDLE);
/* This trick doesn't work apparently because the translate hooks
are single shot. If the core_hook returns OK, then our hook is
not called.
ap_hook_translate_name(file_cache_xlat, aszPre, NULL, APR_HOOK_MIDDLE);
*/


}

but in mod_userdir.c :

static void register_hooks(apr_pool_t *p)
{
    static const char * const aszPre[]={ "mod_alias.c",NULL };
    static const char * const aszSucc[]={ "mod_vhost_alias.c",NULL };


ap_hook_translate_name(translate_userdir,aszPre,aszSucc,APR_HOOK_MIDDLE); #ifdef HAVE_UNIX_SUEXEC

ap_hook_get_suexec_identity(get_suexec_id_doer,NULL,NULL,APR_HOOK_MIDDLE);
#endif
}


I'll commit the following (ok ?) :



static void jk2_register_hooks(apr_pool_t *p) { static const char * const aszPre[]={ "mod_rewrite.c",NULL };

ap_hook_handler(jk2_handler, NULL, NULL, APR_HOOK_MIDDLE);
ap_hook_post_config(jk2_post_config,NULL,NULL,APR_HOOK_MIDDLE);
/* Force the mpm to run before us and set the scoreboard image */
ap_hook_child_init(jk2_child_init,NULL,NULL,APR_HOOK_LAST);
ap_hook_translate_name(jk2_translateaszPre,NULL,APR_HOOK_FIRST);
ap_hook_map_to_storage(jk2_map_to_storage, NULL, NULL, APR_HOOK_MIDDLE);
}




---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]



Reply via email to