opendev/gerrit
Code Issues Proposed changes

Fix getStringList in GerritConfig wrapper class

Browse Source 
When getList from secure store class returns empty list try to get value
from gerrit.config. JGit getStringList will return empty list for not
found value therefore it is safe to ask base config file for this
property.

Change-Id: Iee8b6256a5ab7a1de698c46d2415a5a272f7e0c0
Signed-off-by: Dariusz Luksza <dariusz@luksza.org>
This commit is contained in:
Dariusz Luksza 2014-09-12 10:09:16 +02:00
parent d9b8b39088
commit 0f8fcab555
1 changed files with 1 additions and 1 deletions

2
gerrit-server/src/main/java/com/google/gerrit/server/config/GerritConfig.java

@ -38,7 +38,7 @@ class GerritConfig extends Config {
@Override
public String[] getStringList(String section, String subsection, String name) {
String[] secure = secureStore.getList(section, subsection, name);
if (secure != null) {
if (secure != null && secure.length > 0) {
return secure;
}
return super.getStringList(section, subsection, name);