From 34d1fdb4fe85d1297b7160fcce824a552031daae Mon Sep 17 00:00:00 2001 From: James Moger Date: Wed, 17 Jul 2013 10:07:18 -0400 Subject: [PATCH] Fixed bug with adding new local users with external authentication --- releases.moxie | 3 ++- src/main/java/com/gitblit/GitBlit.java | 11 ++++++----- 2 files changed, 8 insertions(+), 6 deletions(-) diff --git a/releases.moxie b/releases.moxie index f2a27b7c..2e3daf58 100644 --- a/releases.moxie +++ b/releases.moxie @@ -14,8 +14,9 @@ r18: { - Fixed missing model class dependencies in Gitblit Manager build - Fix for IE10 compatability mode - Reset dashboard and activity commit cache on branch REWIND or DELETE + - Fixed bug with adding new local users with external authentication changes: - - updated Chinese translation + - updated Chinese translation additions: ~ dependencyChanges: ~ contributors: diff --git a/src/main/java/com/gitblit/GitBlit.java b/src/main/java/com/gitblit/GitBlit.java index 301591f5..35ef7708 100644 --- a/src/main/java/com/gitblit/GitBlit.java +++ b/src/main/java/com/gitblit/GitBlit.java @@ -84,6 +84,7 @@ import org.slf4j.LoggerFactory; import com.gitblit.Constants.AccessPermission; import com.gitblit.Constants.AccessRestrictionType; +import com.gitblit.Constants.AccountType; import com.gitblit.Constants.AuthenticationType; import com.gitblit.Constants.AuthorizationControl; import com.gitblit.Constants.FederationRequest; @@ -695,12 +696,12 @@ public class GitBlit implements ServletContextListener { public boolean supportsCredentialChanges(UserModel user) { if (user == null) { return false; - } else if (!Constants.EXTERNAL_ACCOUNT.equals(user.password)) { - // credentials likely maintained by Gitblit - return userService.supportsCredentialChanges(); + } else if (AccountType.LOCAL.equals(user.accountType)) { + // local account, we can change credentials + return true; } else { - // credentials are externally maintained - return false; + // external account, ask user service + return userService.supportsCredentialChanges(); } } -- 2.39.5