gromov/casdoor
1
0
Fork 0
forked from casdoor/casdoor
Code Pull Requests Actions Packages Releases Activity

Fix WeChat avatar display error by uploading OAuth avatars to permanent CDN storage

Browse Source 
Co-authored-by: hsluoyz <3787410+hsluoyz@users.noreply.github.com>
This commit is contained in:
copilot-swe-agent[bot]
2026-03-18 10:29:25 +00:00
parent 406f0d3792
commit 33eb4b1d2d
1 changed files with 19 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

20
object/user_util.go
View File

@@ -250,10 +250,28 @@ func SetUserOAuthProperties(organization *Organization, user *User, providerType
if userInfo.AvatarUrl != "" {
propertyName := fmt.Sprintf("oauth_%s_avatarUrl", providerType)
oldAvatarUrl := getUserProperty(user, propertyName)
setUserProperty(user, propertyName, userInfo.AvatarUrl)
if user.Avatar == "" || user.Avatar == organization.DefaultAvatar {
if user.Avatar == "" || user.Avatar == organization.DefaultAvatar || user.Avatar == oldAvatarUrl {
user.Avatar = userInfo.AvatarUrl
}
// Re-upload to permanent storage when OAuth avatar URL changes or permanent avatar is not set.
// This prevents display issues when the temporary OAuth avatar URL (e.g. WeChat) expires.
if oldAvatarUrl != userInfo.AvatarUrl || user.PermanentAvatar == "" {
permanentAvatarUrl, err := getPermanentAvatarUrl(user.Owner, user.Name, userInfo.AvatarUrl, true)
if err != nil {
return false, err
}
if permanentAvatarUrl != "" {
user.PermanentAvatar = permanentAvatarUrl
// Update user.Avatar to the permanent CDN URL if it is currently
// using the (possibly temporary) OAuth avatar URL.
if user.Avatar == userInfo.AvatarUrl {
user.Avatar = permanentAvatarUrl
}
}
}
}
// Apply custom user mapping from provider configuration