From a5a6aba7c80618a45bb57616c7378917675cadab Mon Sep 17 00:00:00 2001 From: kolaente Date: Mon, 25 Dec 2023 17:09:19 +0100 Subject: [PATCH] fix(auth): don't reset user settings when updating name or email from external auth provider Resolves https://github.com/go-vikunja/api/issues/102 --- pkg/modules/auth/openid/openid.go | 9 ++------- 1 file changed, 2 insertions(+), 7 deletions(-) diff --git a/pkg/modules/auth/openid/openid.go b/pkg/modules/auth/openid/openid.go index 1ec91f12585..c7facabbdfd 100644 --- a/pkg/modules/auth/openid/openid.go +++ b/pkg/modules/auth/openid/openid.go @@ -261,13 +261,8 @@ func getOrCreateUser(s *xorm.Session, cl *claims, issuer, subject string) (u *us if cl.Name != u.Name { u.Name = cl.Name } - u, err = user.UpdateUser(s, &user.User{ - ID: u.ID, - Email: u.Email, - Name: u.Name, - Issuer: issuer, - Subject: subject, - }, false) + + u, err = user.UpdateUser(s, u, false) if err != nil { return nil, err }