Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[BUG] I've been trying to use the update_user_info REST API to remove the user's profile picture, but it doesn't seem to be working. #2514

Closed
cipchk opened this issue Aug 12, 2024 · 5 comments · Fixed by #2552
Assignees
Labels
bug Categorizes issue or PR as related to a bug.

Comments

@cipchk
Copy link

cipchk commented Aug 12, 2024

OpenIM Server Version

3.8.0

Operating System and CPU Architecture

Linux (AMD)

Deployment Method

Docker Deployment

Bug Description and Steps to Reproduce

当用户删除头像时,调用 apis/userManagement/updateUserInfo 无法去除,以下两种请求参数都无法做到清除用户头像:

{"userInfo":{"userID":"25","nickname":"Me","faceURL":""}}

{"userInfo":{"userID":"25","nickname":"Me","faceURL":null}}

此外,这一现象在群组头像也是如此。

Screenshots Link

No response

@cipchk cipchk added the bug Categorizes issue or PR as related to a bug. label Aug 12, 2024
@skiffer-git skiffer-git changed the title [BUG] update_user_info REST 接口无法去除用户头像 [BUG] I've been trying to use the update_user_info REST API to remove the user's profile picture, but it doesn't seem to be working. Aug 13, 2024
@mo3et
Copy link
Contributor

mo3et commented Aug 23, 2024

thanks for your feedback.

user_user_info is deprecated. You need use user/user_user_info_ex.

About GroupInfo update. I solve this problem to #2552. You can use group/set_group_info_ex when this PR is merged.
group/set_group_info_ex the parameters passed is same to group/set_group_info. We also update latest docs when PR is merged.

Thank you again for your feedback, and we would also welcome you to become one of our contributors.

@cipchk
Copy link
Author

cipchk commented Aug 23, 2024

I tried using user/update_user_info_ex, but it still failed; I’m using the version OPENIM_SERVER_IMAGE=openim/openim-server:release-v3.8.

@mo3et
Copy link
Contributor

mo3et commented Aug 26, 2024

hello, i tried this interface in the server. But is work it.

image

my avator is truely remove.

can you upload more logs?

@mo3et mo3et reopened this Aug 26, 2024
@cipchk
Copy link
Author

cipchk commented Aug 27, 2024

Sorry, my bad. I passed a null instead of an empty string. It's working now.

@cipchk cipchk closed this as completed Aug 27, 2024
@mo3et
Copy link
Contributor

mo3et commented Aug 27, 2024

Sorry, my bad. I passed a null instead of an empty string. It's working now.

Passed a “” is correct. This interface use pointer pass fields. This operate avoid zero Value ignore.

Now set_group_info_ex is available in sourceCode deployment. I will update docs soon.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Categorizes issue or PR as related to a bug.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants