From 06116dd5f166ec2fff0531d21d6a291c9018703c Mon Sep 17 00:00:00 2001 From: James Moger <james.moger@gitblit.com> Date: Mon, 17 Jun 2013 16:00:17 -0400 Subject: [PATCH] Minor tweaks to SalesForce user service --- src/main/java/com/gitblit/GitBlit.java | 15 +++++++++++++-- 1 files changed, 13 insertions(+), 2 deletions(-) diff --git a/src/main/java/com/gitblit/GitBlit.java b/src/main/java/com/gitblit/GitBlit.java index 25ffaba..ecd4662 100644 --- a/src/main/java/com/gitblit/GitBlit.java +++ b/src/main/java/com/gitblit/GitBlit.java @@ -681,7 +681,15 @@ * @return true if the user service supports credential changes */ public boolean supportsCredentialChanges(UserModel user) { - return (user != null && user.isLocalAccount()) || userService.supportsCredentialChanges(); + if (user == null) { + return false; + } else if (!Constants.EXTERNAL_ACCOUNT.equals(user.password)) { + // credentials likely maintained by Gitblit + return userService.supportsCredentialChanges(); + } else { + // credentials are externally maintained + return false; + } } /** @@ -921,7 +929,10 @@ if (userService == null) { return; } - if (userService.supportsCookies()) { + GitBlitWebSession session = GitBlitWebSession.get(); + boolean standardLogin = session.authenticationType.isStandard(); + + if (userService.supportsCookies() && standardLogin) { Cookie userCookie; if (user == null) { // clear cookie for logout -- Gitblit v1.9.1