-
Notifications
You must be signed in to change notification settings - Fork 2.5k
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] When calling getAdvancedHistoryMessageList
in a group, it generates a large number of get_users_info
requests
#2831
Comments
@icey-yu 有任何进展吗?因为这个情况导致我们现在都不敢让所有用户都使用 3.8.1 版本。 |
@icey-yu Any progress? Because of this situation, we now dare not let all users use version 3.8.1. |
We are investigating the issue. |
In our tests, the SDK's interface calls are functioning normally. The logs you filtered may include entries from other calls. You can use the operationID to confirm if it’s from the same call. |
@icey-yu Yes, they all share the same operationid, and I can confirm that it gets generated when calling Moreover, this issue is not limited to the Flutter side; it also applies to our Web side. |
Is the conversation a group chat, and are most participants in the group non-friends? |
@icey-yu Yes, over 99% of them are not friends with each other. I created the group through https://doc.rentsoft.cn/zh-Hans/restapi/apis/groupManagement/createGroup, and all members were generated randomly. You can easily run the test. |
We optimized this issue a few days ago. You can pull the latest code from the main branch of https://github.com/openimsdk/openim-sdk-core and try again. |
Could you provide a specific PR related to optimization? |
OpenIM Server Version
3.8.1
Operating System and CPU Architecture
Linux (AMD)
Deployment Method
Docker Deployment
Bug Description and Steps to Reproduce
Here is the request code:
Requesting 20 records generates more than 40
get_users_info
requests, see PNG:This is the specific log of one of the requests and responses, and there are over 40 similar ones. The only difference is that they all have different userIDs:
Screenshots Link
No response
The text was updated successfully, but these errors were encountered: