Migrate away from apache access_compat to authz_host. #73

Open
bwaegeneire wants to merge 1 commit from apache-require into master
Owner

Since apache 2.4 the directives Allow, Order, Deny and Satisfy are deprecated. This pull request replace them with Require directives.

Since apache [2.4](https://httpd.apache.org/docs/2.4/en/upgrading.html) the directives Allow, Order, Deny and Satisfy are deprecated. This pull request replace them with Require directives.
bwaegeneire added 1 commit 2021-02-01 11:19:11 +01:00
bwaegeneire force-pushed apache-require from bc82c5a67f to 07a540cc8c 2021-09-22 10:35:55 +02:00 Compare
mtrossevin requested changes 2023-11-17 14:51:04 +01:00
mtrossevin left a comment
Owner

I think you forgot one directive (see below).

I think you forgot one directive (see below).
mtrossevin requested changes 2023-11-17 14:53:48 +01:00
mtrossevin left a comment
Owner

I think you forgot one directive (see below).

I think you forgot one directive (see below).
@ -33,3 +32,1 @@
Order Deny,Allow
Deny from all
Require valid-user
Owner

Shouldn't it also require a Require all denied or to be in a <RequireAll></RequireAll> ? I'm not certain on if Require valid-user generate a negative or neutral result if it isn't matched.

Shouldn't it also require a `Require all denied` or to be in a `<RequireAll></RequireAll>` ? I'm not certain on if `Require valid-user` generate a negative or neutral result if it isn't matched.
This pull request has changes requested by an official reviewer.
This branch is out-of-date with the base branch
You are not authorized to merge this pull request.

Checkout

From your project repository, check out a new branch and test the changes.
git fetch -u origin apache-require:apache-require
git checkout apache-require
Sign in to join this conversation.
No description provided.