You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@httpd.apache.org by André Malo <nd...@perlig.de> on 2007/09/02 15:41:52 UTC

Re: svn commit: r571872 - in /httpd/httpd/trunk: CHANGES docs/manual/mod/mod_include.xml modules/filters/mod_include.c modules/filters/mod_include.h

* minfrin@apache.org wrote:

> Author: minfrin
> Date: Sat Sep  1 14:27:26 2007
> New Revision: 571872
>
> URL: http://svn.apache.org/viewvc?rev=571872&view=rev
> Log:
> mod_include: Add an "if" directive syntax to test whether an URL
> is accessible, and if so, conditionally display content. This
> allows a webmaster to hide a link to a private page when the user
> has no access to that page.

> Modified: httpd/httpd/trunk/modules/filters/mod_include.h
> URL:
> http://svn.apache.org/viewvc/httpd/httpd/trunk/modules/filters/mod_includ
>e.h?rev=571872&r1=571871&r2=571872&view=diff
> =========================================================================
>===== --- httpd/httpd/trunk/modules/filters/mod_include.h (original)
> +++ httpd/httpd/trunk/modules/filters/mod_include.h Sat Sep  1 14:27:26
> 2007 @@ -96,6 +96,9 @@
>
>      /* pointer to internal (non-public) data, don't touch */
>      struct ssi_internal_ctx *intern;
> +
> +    /* is using the access tests allowed? */
> +    int         accessenable;
>  } include_ctx_t;


Is there a particular reason, that you changed the public context instead of 
the internal (private) one?

nd
-- 
die (eval q-qq:Just Another Perl Hacker
:-)

# André Malo, <http://pub.perlig.de/> #

Re: svn commit: r571872 - in /httpd/httpd/trunk: CHANGES docs/manual/mod/mod_include.xml modules/filters/mod_include.c modules/filters/mod_include.h

Posted by André Malo <nd...@perlig.de>.
* Graham Leggett wrote:

> André Malo wrote:
> > Is there a particular reason, that you changed the public context
> > instead of the internal (private) one?
>
> Because I was under the incorrect impression that all the other
> directives were being written there. I've changed it - thanks for
> catching it out.

Ah, was just wondering. Thanks.

nd
-- 
"Umfassendes Werk (auch fuer Umsteiger vom Apache 1.3)"
                                          -- aus einer Rezension

<http://pub.perlig.de/books.html#apache2>

Re: svn commit: r571872 - in /httpd/httpd/trunk: CHANGES docs/manual/mod/mod_include.xml modules/filters/mod_include.c modules/filters/mod_include.h

Posted by Graham Leggett <mi...@sharp.fm>.
André Malo wrote:

> Is there a particular reason, that you changed the public context instead of 
> the internal (private) one?

Because I was under the incorrect impression that all the other 
directives were being written there. I've changed it - thanks for 
catching it out.

Regards,
Graham
--