opendev/gerrit
Code Issues Proposed changes

Enable Git protocol v2 unconditionally

Browse Source 
We also don't have corresponding guards for gerrit for protocols v0 and
v1. The original guard was also implemented because Git protocol v2
support was a late feature in 2.16 release and we were not confident
that it works as expected. This change is done on master, and we have
enough time to test it until the first release candidate is cut.

Change-Id: I07c9061d7a364a6e34ec889e2f9d39a38295935d
This commit is contained in:
David Ostrovsky
2019-01-13 11:15:50 +01:00
committed by Luca Milanesio
parent fb64381f7c
commit b345f0a208
4 changed files with 5 additions and 32 deletions
Download Patch File Download Diff File Expand all files Collapse all files

6
java/com/google/gerrit/server/git/TransferConfig.java
View File

@@ -29,7 +29,6 @@ public class TransferConfig {
private final long maxObjectSizeLimit;
private final String maxObjectSizeLimitFormatted;
private final boolean inheritProjectMaxObjectSizeLimit;
private final boolean enableProtocolV2;
@Inject
TransferConfig(@GerritServerConfig Config cfg) {
@@ -46,7 +45,6 @@ public class TransferConfig {
maxObjectSizeLimitFormatted = cfg.getString("receive", null, "maxObjectSizeLimit");
inheritProjectMaxObjectSizeLimit =
cfg.getBoolean("receive", "inheritProjectMaxObjectSizeLimit", false);
enableProtocolV2 = cfg.getBoolean("receive", "enableProtocolV2", false);
packConfig = new PackConfig();
packConfig.setDeltaCompress(false);
@@ -74,8 +72,4 @@ public class TransferConfig {
public boolean inheritProjectMaxObjectSizeLimit() {
return inheritProjectMaxObjectSizeLimit;
}
public boolean enableProtocolV2() {
return enableProtocolV2;
}
}