diff --git a/app/login.go b/app/login.go index 2cbfcaed70..2ee7a6289a 100644 --- a/app/login.go +++ b/app/login.go @@ -217,8 +217,10 @@ func (a *App) DoLogin(w http.ResponseWriter, r *http.Request, user *model.User, a.SetSession(session) if a.Srv().License() != nil && *a.Srv().License().Features.LDAP && a.Ldap() != nil { + userVal := *user + sessionVal := *session a.Srv().Go(func() { - a.Ldap().UpdateProfilePictureIfNecessary(*user, session) + a.Ldap().UpdateProfilePictureIfNecessary(userVal, sessionVal) }) } diff --git a/einterfaces/ldap.go b/einterfaces/ldap.go index 3784251a08..fb91792eec 100644 --- a/einterfaces/ldap.go +++ b/einterfaces/ldap.go @@ -21,6 +21,6 @@ type LdapInterface interface { GetGroup(groupUID string) (*model.Group, *model.AppError) GetAllGroupsPage(page int, perPage int, opts model.LdapGroupSearchOpts) ([]*model.Group, int, *model.AppError) FirstLoginSync(user *model.User, userAuthService, userAuthData, email string) *model.AppError - UpdateProfilePictureIfNecessary(model.User, *model.Session) + UpdateProfilePictureIfNecessary(model.User, model.Session) GetADLdapIdFromSAMLId(authData string) string } diff --git a/einterfaces/mocks/LdapInterface.go b/einterfaces/mocks/LdapInterface.go index 4389ce3776..a2d2c3103c 100644 --- a/einterfaces/mocks/LdapInterface.go +++ b/einterfaces/mocks/LdapInterface.go @@ -305,6 +305,6 @@ func (_m *LdapInterface) SwitchToLdap(userId string, ldapId string, ldapPassword } // UpdateProfilePictureIfNecessary provides a mock function with given fields: _a0, _a1 -func (_m *LdapInterface) UpdateProfilePictureIfNecessary(_a0 model.User, _a1 *model.Session) { +func (_m *LdapInterface) UpdateProfilePictureIfNecessary(_a0 model.User, _a1 model.Session) { _m.Called(_a0, _a1) }