Re: Freeze break request: Fix mirrorlist2 static files

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Tue, Apr 14, 2015 at 12:01:52PM +0200, Patrick Uiterwijk wrote:
> -----BEGIN PGP SIGNED MESSAGE-----
> Hash: SHA512
> 
> Hi,
> 
> Currently, static files for Mirrorlist2 give 403 Access Denied's.
> Could I get +1s to apply this fix:
> 
> 
> commit ee955fa452df4c753c276b37a34af04933288e20
> Author: Patrick Uiterwijk <puiterwijk@xxxxxxxxxx>
> Date:   Tue Apr 14 09:58:38 2015 +0000
> 
>     Add a Location tag for static
>     
>     For every alias, you need to have either a Directory or a Location
>     that matches it.
>     Source: https://wiki.apache.org/httpd/ClientDeniedByServerConfiguration
>     
>     Upstream PR: https://github.com/fedora-infra/mirrormanager2/pull/41
>     
>     Signed-off-by: Patrick Uiterwijk <puiterwijk@xxxxxxxxxx>
> 
> diff --git a/roles/mirrormanager/mirrorlist2/templates/mirrorlist-server.conf b/roles/mirrormanager/mirrorlist2/templa
> index 91800c5..adb5182 100644
> - --- a/roles/mirrormanager/mirrorlist2/templates/mirrorlist-server.conf
> +++ b/roles/mirrormanager/mirrorlist2/templates/mirrorlist-server.conf
> @@ -52,3 +52,15 @@ WSGIScriptAlias /mirrorlist /usr/share/mirrormanager2/mirrorlist_client.wsgi
>          Allow from all
>      </IfModule>
>  </Location>
> +
> +<Location /static>
> +    <IfModule mod_authz_core.c>
> +        # Apache 2.4
> +        Require all granted
> +    </IfModule>
> +    <IfModule !mod_authz_core.c>
> +        # Apache 2.2
> +        Order deny,allow
> +        Allow from all
> +    </IfModule>
> +</Location>

I'm not too happy with this because we are putting MM1 related code into MM2,
but since it'll fix the current situation +1 for now


Pierre

Attachment: pgpjgAjiRaOOF.pgp
Description: PGP signature

_______________________________________________
infrastructure mailing list
infrastructure@xxxxxxxxxxxxxxxxxxxxxxx
https://admin.fedoraproject.org/mailman/listinfo/infrastructure

[Index of Archives]     [Fedora Development]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Yosemite News]     [KDE Users]

  Powered by Linux