Evolution #1157
fermé
Mise à jour de l'exemple de configuration nginx
100%
Description
La configuration d'exemple de la documentation pour nginx ne respecte pas les bonnes pratiques et est daté.
Je joins un patch pour la mettre à jour.
Fichiers
Mis à jour par Adrien Pachkoff il y a presque 7 ans
- Fichier Mise-jour-de-l-exemple-de-configuration-nginx(1).patch Mise-jour-de-l-exemple-de-configuration-nginx(1).patch ajouté
Oups ! Il n'y avait plus de réécriture des URLs avec cette version. Je joins un patch correct à la place, cette fois. :D
Mis à jour par Johan Cwiklinski il y a presque 7 ans
- Statut changé de Nouveau à In Progress
- Assigné à mis à Johan Cwiklinski
Merci :)
Concernant index index.html index.php;, index.html me semble inutile.
La configuration SSL doit être ajoutée côté apache aussi (les configurations doivent être équivalentes). Je m'en chargerai.
Bien vu pour le deny sur les différents dossiers ;)
Mis à jour par Johan Cwiklinski il y a presque 7 ans
Concernant la règle de récriture ; elle ne devrait entrer en action que si le fichier/dossier demandé n'existe pas... Du coup, j ne comprend pas pourquoi if (!-e $request_filename){ a été supprimé.
Mis à jour par Johan Cwiklinski il y a presque 7 ans
Voir https://github.com/galette/documentation/compare/master...hotfix/0.9005 pour suivre les modifications.
Mis à jour par Adrien Pachkoff il y a presque 7 ans
Johan Cwiklinski a écrit :
Concernant la règle de récriture ; elle ne devrait entrer en action que si le fichier/dossier demandé n'existe pas... Du coup, j ne comprend pas pourquoi
if (!-e $request_filename){a été supprimé.
C'est exactement ce à quoi sert `try_files` https://www.nginx.com/resources/wiki/start/topics/depth/ifisevil/
Ici, on teste si il existe un fichier avec le nom demandé, ou un répertoire, et sinon on passe à la "location" `@galette`.
Mis à jour par Johan Cwiklinski il y a presque 7 ans
Adrien Pachkoff a écrit :
Ici, on teste si il existe un fichier avec le nom demandé, ou un répertoire, et sinon on passe à la "location" `@galette`.
OK, merci pour la précision :)
Mis à jour par Anonyme il y a presque 7 ans
- Statut changé de In Progress à Résolu
- % réalisé changé de 0 à 100
Appliqué par commit 6bf5d275ee8b874693f6b60a61bc05acf808af52.