Peng Yu <pengyu...@gmail.com> writes: > I was looking at the patched version. (See the patch posted in bug > #31147) So I think that the bug in the patch (see the relevant code > below, where full_file has the query string). I guess for full_file a > different 'acceptable' function should be used. > > if (opt.match_query_string) full_file = concat_strings(u->file, "?", > u->query, (char *) 0); > > if (!acceptable (full_file)) > { > DEBUGP (("%s (%s) does not match acc/rej rules.\n", > url, full_file)); > goto out; > } > }
I am inclined to don't add more options to the current Accept/Reject rules, as I think they are not flexible enough and quite tricky. It is better to support a more generic way to specify these rules. Cheers, Giuseppe