BBlack has submitted this change and it was merged. Change subject: LE: comment out most of apache challenge conf ......................................................................
LE: comment out most of apache challenge conf Turns out having the Require statement at all starts pulling in Auth stuff. This config works for the rt.wm.o case, we'll see how it goes with others as we go! Change-Id: I869aa9a58aea3b599efc472252aee76d46d502b1 --- M modules/letsencrypt/files/challenge-apache.conf 1 file changed, 5 insertions(+), 3 deletions(-) Approvals: BBlack: Verified; Looks good to me, approved diff --git a/modules/letsencrypt/files/challenge-apache.conf b/modules/letsencrypt/files/challenge-apache.conf index 2546b28..566bc2e 100644 --- a/modules/letsencrypt/files/challenge-apache.conf +++ b/modules/letsencrypt/files/challenge-apache.conf @@ -1,4 +1,6 @@ Alias "/.well-known/acme-challenge" "/var/acme/challenge" -<Directory "/var/acme/challenge"> - Require all granted -</Directory> +# Just the Alias line for now, works for rt.wm.o (first test case) +# It may be that every unique apache config will need different things... +#<Directory "/var/acme/challenge"> +# Require all granted +#</Directory> -- To view, visit https://gerrit.wikimedia.org/r/285577 To unsubscribe, visit https://gerrit.wikimedia.org/r/settings Gerrit-MessageType: merged Gerrit-Change-Id: I869aa9a58aea3b599efc472252aee76d46d502b1 Gerrit-PatchSet: 1 Gerrit-Project: operations/puppet Gerrit-Branch: production Gerrit-Owner: BBlack <bbl...@wikimedia.org> Gerrit-Reviewer: BBlack <bbl...@wikimedia.org> _______________________________________________ MediaWiki-commits mailing list MediaWiki-commits@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits
