diff options
author | James Moger <james.moger@gitblit.com> | 2013-09-23 10:55:19 -0400 |
---|---|---|
committer | James Moger <james.moger@gitblit.com> | 2013-09-23 10:55:19 -0400 |
commit | b2d7f418bf289e5fa6149ac1b5744cfb03b60dea (patch) | |
tree | 0457dae61d683348c56b79dcacf32eb6d9ee64bc /src/main/java/com/gitblit/ConfigUserService.java | |
parent | bb79224f6b59d71098863ada728623bd728df6ff (diff) | |
download | gitblit-b2d7f418bf289e5fa6149ac1b5744cfb03b60dea.tar.gz gitblit-b2d7f418bf289e5fa6149ac1b5744cfb03b60dea.zip |
Eliminate redundant read() and use accessor instead of direct cache access
Diffstat (limited to 'src/main/java/com/gitblit/ConfigUserService.java')
-rw-r--r-- | src/main/java/com/gitblit/ConfigUserService.java | 4 |
1 files changed, 1 insertions, 3 deletions
diff --git a/src/main/java/com/gitblit/ConfigUserService.java b/src/main/java/com/gitblit/ConfigUserService.java index 2475b9ab..a9217b5c 100644 --- a/src/main/java/com/gitblit/ConfigUserService.java +++ b/src/main/java/com/gitblit/ConfigUserService.java @@ -186,8 +186,7 @@ public class ConfigUserService implements IUserService { if (!StringUtils.isEmpty(model.cookie)) {
return model.cookie;
}
- read();
- UserModel storedModel = users.get(model.username.toLowerCase());
+ UserModel storedModel = getUserModel(model.username);
if (storedModel == null) {
return null;
}
@@ -229,7 +228,6 @@ public class ConfigUserService implements IUserService { */
@Override
public UserModel authenticate(String username, char[] password) {
- read();
UserModel returnedUser = null;
UserModel user = getUserModel(username);
if (user == null) {
|