|
Hi Bill (this is a resend to include the dev and user communities, per your instructions): Thanks very much for the prompt response. I believe we have covered all of the steps you indicate below. Attached please find a tar file that contains a simple
C++ module and a makefile to build it. Our test server http.conf has a LoadModule line for the module and then the following set of directory sections: <Directory /foo> dirConfig /foo </Directory> <Directory /> dirConfig / </Directory> <Directory /foo/bar> dirConfig /foo/bar </Directory> When we execute the server, we see this on the console: # bin/apachectl start Created dirConfig for context: unset Created dirConfig for context: /foo/ Inserted dirConfig with message: /foo Created dirConfig for context: / Inserted dirConfig with message: / Created dirConfig for context: /foo/bar/ Inserted dirConfig with message: /foo/bar Created dirConfig for context: unset And this in the log: Merged config: unset:/ … hookFunc: for request /foo/bar/foo.html the context is: unset:/ We do not see any other messages. The file /foo/bar/foo.html exists and the browser displays it. This suggests that the merge sequence for the last directory
section stopped with the configuration for the “/” section. Perhaps you could point out what would cause this behavior? Thanks: Adam From: William A. Rowe Jr. [mailto:wrowe@xxxxxxxxxx]
Hi Jin, there might be more than one thing going on here. First, it is critical that a directive belonging to the module occurs in each of the <section > blocks you are merging. Remember httpd is not going to even create a config section, never mind merge them, for every module whose directives do not appear in a given <section > - this is what makes httpd so efficient. Second, if there is a bug in the cmd handler, your ctx member might not be correctly updated for a section, same is true for a bug in the create or merge function. Third, httpd does perform some optimization, it may premerge global configs and may resume a merge from previously merged sections when they are encountered in a subrequest. Is the resulting ->ctx member correct for the resulting <Directory > context? Is it simply that the merge isn't called as often as expected? Optimization may be the cause. Is the cmd record for your directive set to OR_ACCESS (telling httpd that it is a per-dir and not per-server config?) Is there a bug in your create code that is returning NULL instead of a newly initialized config section? If we look at the example of mod_dir.c, here are the key points... AP_DECLARE_MODULE(dir) = {
STANDARD20_MODULE_STUFF,
create_dir_config, /* create per-directory config structure */
merge_dir_configs, /* merge per-directory config structures */
All is well, we have a create + merge handler... static void *create_dir_config(apr_pool_t *p, char *dummy)
{
dir_config_rec *new = apr_pcalloc(p, sizeof(dir_config_rec));
new->index_names = NULL;
new->do_slash = MODDIR_UNSET;
new->checkhandler = MODDIR_UNSET;
new->redirect_index = REDIRECT_UNSET;
return (void *) new;
}
the correct structure size is created and members initialized to empty (e.g. 'unset') - the new allocation is returned. static void *merge_dir_configs(apr_pool_t *p, void *basev, void *addv)
{
dir_config_rec *new = apr_pcalloc(p, sizeof(dir_config_rec));
dir_config_rec *base = (dir_config_rec *)basev;
dir_config_rec *add = (dir_config_rec *)addv;
new->index_names = add->index_names ? add->index_names : base->index_names;
new->do_slash =
(add->do_slash == MODDIR_UNSET) ? base->do_slash : add->do_slash;
new->checkhandler =
(add->checkhandler == MODDIR_UNSET) ? base->checkhandler : add->checkhandler;
new->redirect_index=
(add->redirect_index == REDIRECT_UNSET) ? base->redirect_index : add->redirect_index;
new->dflt = add->dflt ? add->dflt : base->dflt;
return new;
}
A new config is created, the various per-dir values updated, and the resulting new allocation is returned. static const command_rec dir_cmds[] =
{
...
AP_INIT_RAW_ARGS("DirectoryIndex", add_index, NULL, DIR_CMD_PERMS,
"a list of file names"),
The DIR_CMD_PERMS (defined as OR_INDEXES) assures httpd that this is a per-dir config directive allowed wherever the 'AllowOverride Indexes' is set. static const char *add_index(cmd_parms *cmd, void *dummy, const char *arg)
{
dir_config_rec *d = dummy;
const char *t, *w;
int count = 0;
if (!d->index_names) {
d->index_names = apr_array_make(cmd->pool, 2, sizeof(char *));
}
t = arg;
while ((w = ap_getword_conf(cmd->pool, &t)) && w[0]) {
if (count == 0 && !strcasecmp(w, "disabled")) {
/* peek to see if "disabled" is first in a series of arguments */
const char *tt = t;
const char *ww = ap_getword_conf(cmd->temp_pool, &tt);
if (ww[0] == '\0') {
/* "disabled" is first, and alone */
apr_array_clear(d->index_names);
break;
}
}
*(const char **)apr_array_push(d->index_names) = w;
count++;
}
return NULL;
}
Here we iterate the string (RAW_ARGS) passed by DirectoryIndex and parse it to build an array of index strings, and see where the dir_config_rec is derived from the dummy arg (it would be given a newly/recently
created dir config). Note also that a second occurance of DirectoryIndex keeps building this <section >'s index_name list. Back up in the merge either keep the parent list, or replace it if a new list was created with the add_index directive. Please check some of the items identified above to see if you can work out the cause of your bug. Yours, Bill [FYI I do ignore personal appeals for help at my @apache.org account, kindly requesting that the user restate their question publicly on the users@ or dev@ list as
appropriate. This lets me answer the same question for many end users who google for relevant keywords of the question or answer. For private inquiries about httpd module authoring under your support engagement, please continue to use
wrowe@xxxxxxxxxx, but I'm certainly happy to publicly answer questions if you like - this becomes interesting when there is a probable design defect for the entire dev@ team to consider.] On Thu, Oct 29, 2015 at 2:32 PM, PAN, JIN <Jin.Pan@xxxxxxx> wrote:
Hi Bill,
This is Jin from Fidelity, new Hampshire campus. Very sorry about the double emails, I wanted to make sure this email can get to you from all angles. We’d greatly appreciate any comments on this problem we are perplexed with.
The issue is regarding the Apache <Directory> merging. We want to construct a breadcrumb trail from multiple <Directory> contexts for each request as below. We tried to use the directory merge function to concatenate different context as the merge function
is invoked by Apache as it walks to every <Directory> block. We observed that Apache does not provide the BASE and NEW pointer content to the merge hook for <Directory>, so that at the end of the last <Directory> block, we are unable to get a combined “/:/foo:/foo/bar”
trail result. However, the same merge hook does get the BASE and NEW pointer content for <Location> blocks, for which we do see the desired combination trail after all <Location> blocks.
Question is, is there any way to merge the different <Directory> context together?
Config:
<Directory /foo>
Header merge someHeader "/foo"
ß mod_headers seems to suffer the same, no BASE or NEW to merge with
dirConfig /foo
</Directory>
<Directory />
Header merge someHeader "/"
dirConfig /
</Directory>
<Directory /foo/bar>
Header merge someHeader "/foo/bar"
dirConfig /foo/bar
</Directory>
Some code snippet to explain what we’re doing…
AP_DECLARE_MODULE(dirconfig) =
{
STANDARD20_MODULE_STUFF,
create_dir,
merge_dir,
NULL,
NULL,
cmds,
register_hooks
};
typedef struct
{
char* cxt;
} scfg;
static void *merge_dir(apr_pool_t *pool, void *BASE, void *NEW)
{
scfg *m = apr_pcalloc(pool, sizeof(*m));
scfg *p = BASE;
ßpoints to the proper “/foo/bar” content if in <Location> but empty if in <Directory> Why?
scfg *c = NEW;
m->cxt = apr_pstrcat(pool, p->cxt, ":", c->cxt, NULL);
return m;
}
Thank you!
Jin |
Attachment:
dir-config-tar
Description: dir-config-tar
--------------------------------------------------------------------- To unsubscribe, e-mail: users-unsubscribe@xxxxxxxxxxxxxxxx For additional commands, e-mail: users-help@xxxxxxxxxxxxxxxx
|