search

GeotrekCE / Geotrek-admin

Paths management for National Parks and Tourism organizations
https://geotrek.fr
BSD 2-Clause "Simplified" License
131 stars 75 forks source link

[DOCUMENTATION] Ajout informations Lets encrypt/NGINX #4233

Open eprouteau opened 2 months ago

eprouteau commented 2 months ago

Bonjour,

Lors de la configuration du certificat SSL via cerbot, nous nous sommes rendu compte que le bloc de la redirection forcée en https incluais la variable $host et que en l'état celle-ci était supprimée lors du reconfigure de geotrek-admin si nous ne mettions pas ${DOLLAR} à la place.

Il pourrait être intéressant d'ajouter cette information dans le bloc SSL de la doc car nous c'est poser la question quelques temps avant de comprendre. :)

image

Il est peut être possible de contribuer à la doc ?

Bonne journée,

camillemonchicourt commented 2 months ago

Il faut l'ajouter dans votre surcouche de conf NGINX pour ne pas que ça soit écrasé à chaque mise à jour. C'est indiqué dans la doc ou des tickets il me semble.

eprouteau commented 2 months ago

Oui c'est bien ce que l'on a fait mais dans le fichier de conf nginx.in géré par geotrek. on n'avait pas vu qu'il fallait remplacer les '$host' de certbot par ${DOLLAR}. Mais lors du reconfigure il supprimait le '$host' il interprétait le $

babastienne commented 2 months ago

Bonjour @eprouteau. Merci pour ce ticket, ça permet en effet de mettre en avant le fait que la documentation n'est sûrement pas suffisamment claire sur ce point.

N'hésitez pas à contribuer à la documentation pour l'enrichir. Pour cela je vous invite à ouvrir une pull request ou à nous proposer via ce ticket une modification de la documentation (peut-être ce fichier ?).

Je reste à disposition si besoin d'informations.