ami/tl-vikunja
1
0
Fork 0
Code

fix(auth): don't reset user settings when updating name or email from external auth provider

Browse Source 
Resolves https://github.com/go-vikunja/api/issues/102
This commit is contained in:
kolaente 2023-12-25 17:09:19 +01:00
parent 32611dc2d7
commit a5a6aba7c8
No known key found for this signature in database
GPG Key ID: F40E70337AB24C9B
1 changed files with 2 additions and 7 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

9
pkg/modules/auth/openid/openid.go
View File

@ -261,13 +261,8 @@ func getOrCreateUser(s *xorm.Session, cl *claims, issuer, subject string) (u *us
if cl.Name != u.Name { if cl.Name != u.Name {
u.Name = cl.Name u.Name = cl.Name
} }
u, err = user.UpdateUser(s, &user.User{
ID: u.ID, u, err = user.UpdateUser(s, u, false)
Email: u.Email,
Name: u.Name,
Issuer: issuer,
Subject: subject,
}, false)
if err != nil { if err != nil {
return nil, err return nil, err
} }