https://bz.apache.org/bugzilla/show_bug.cgi?id=59709
--- Comment #5 from Andrey Chernov <[email protected]> --- (In reply to Eric Covener from comment #4) > from geoip2: > > /* we have two entry points, the header_parser hook, right before > * the authentication hook used for Dirctory specific enabled > geoiplookups > * or right before directory rewrite rules. > */ > ap_hook_header_parser(geoip_per_dir, NULL, aszSucc, APR_HOOK_FIRST); > > /* and the servectly wide hook, after reading the request. Perfecly > * suitable to serve serverwide mod_rewrite actions > */ > ap_hook_post_read_request(geoip_post_read_request, NULL, aszSucc, > APR_HOOK_MIDDLE); > > ap_if_walk() should be between these two. In geoip_per_dir(), after not founding per-directory config, it exits from the hook: dcfg = ap_get_module_config(r->per_dir_config, &geoip_module); if (!dcfg) { return DECLINED; } What wrong happens, in case you'll move ap_if_walk() down to APR_HOOK_MIDDLE? -- You are receiving this mail because: You are the assignee for the bug. --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
