Dzahn has submitted this change and it was merged. ( 
https://gerrit.wikimedia.org/r/391865 )

Change subject: Gerrit: Set auth.gitBasicAuth to true
......................................................................


Gerrit: Set auth.gitBasicAuth to true

Note this will break tools that use digest but anyways with the upcoming
2.14 update, digest was removed and basic auth switched on by default.

Also lfs does not support digest either.

As we use https and not http this will not degrade security.

https://gerrit.wikimedia.org/r/Documentation/config-gerrit.html#auth.gitBasicAuth

Bug: T171758
Change-Id: I3385653168d39c5c27a1bb88b275d44134bc1100
---
M modules/gerrit/templates/gerrit.config.erb
1 file changed, 1 insertion(+), 0 deletions(-)

Approvals:
  Chad: Looks good to me, but someone else must approve
  jenkins-bot: Verified
  Dzahn: Looks good to me, approved



diff --git a/modules/gerrit/templates/gerrit.config.erb 
b/modules/gerrit/templates/gerrit.config.erb
index fe28d1b..11ef99b 100644
--- a/modules/gerrit/templates/gerrit.config.erb
+++ b/modules/gerrit/templates/gerrit.config.erb
@@ -2,6 +2,7 @@
     type = LDAP
     cookieSecure = true
     registerUrl = 
https://wikitech.wikimedia.org/w/index.php?title=Special:UserLogin&returnto=Help%3AGetting+Started&type=signup
+    gitBasicAuth = true
     gitBasicAuthPolicy = HTTP
 [cache]
     directory = cache

-- 
To view, visit https://gerrit.wikimedia.org/r/391865
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I3385653168d39c5c27a1bb88b275d44134bc1100
Gerrit-PatchSet: 6
Gerrit-Project: operations/puppet
Gerrit-Branch: production
Gerrit-Owner: Paladox <[email protected]>
Gerrit-Reviewer: Chad <[email protected]>
Gerrit-Reviewer: Dzahn <[email protected]>
Gerrit-Reviewer: jenkins-bot <>

_______________________________________________
MediaWiki-commits mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits

Reply via email to