On Thu, May 05, 2016 at 09:10:21PM +0200, Lars Schneider wrote:
> > +<LocationMatch /smart_headers/>
> > + <RequireAll>
> > + Require expr %{HTTP:x-magic-one} == 'abra'
> > + Require expr %{HTTP:x-magic-two} == 'cadabra'
> > + </RequireAll>
>
> I think "<RequireAll>" depends on mod_authz_core which is only
> available in Apache HTTPD 2.3 or later [1].
>
> Right now the test only checks if Apache version greater 2
> is installed. Should we guard this test with a special version
> check? Or do you see a way to check the magic values without
> "<RequireAll>"?
I think you can get rid of RequireAll with:
Require expr %{HTTP:x-magic-one} == 'abra' && %{HTTP:x-magic-two} == 'cadabra'
But I am also not sure that "expr" existed in Apache 2.2.
I think the older way of checking headers was to do some trickery with
RewriteCond; I tried briefly to make that work when I wrote the test,
but never did (but I'm far from an expert in Apache; the only reason I
have touched it in the last 15 years is for Git's test suite).
The nuclear option would be to put a shell script between Apache and
git-http-backend that checks for those headers (I suspect we'd still
need some magic in the Apache config to pass the headers out in the
environment, though).
-Peff
--
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to [email protected]
More majordomo info at http://vger.kernel.org/majordomo-info.html
