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.

Sign up for GitHub

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

Jump to bottom

Mitigate MemberJoinRequestEvent and BotInvitedJoinGroupRequestEvent identification issue for QQAndroid #402

Closed
wants to merge 1 commit into from
Closed

Mitigate MemberJoinRequestEvent and BotInvitedJoinGroupRequestEvent identification issue for QQAndroid #402

wants to merge 1 commit into from

Conversation

kenvix
Copy link
Contributor

@kenvix kenvix commented Jun 28, 2020

在某些情况下，管理员邀请群Bot加群会被误判为群成员申请加群。此 commit 缓解了此问题。

@}WDFNRGN97HZJELEI%6{BN

@ryoii
Copy link
Member

ryoii commented Jun 28, 2020

这一部分逻辑和退群的逻辑正在着手重新测试和重写。
如果您有测试到比较全的type数据，将十分感谢

@kenvix
Copy link
Contributor Author

kenvix commented Jun 28, 2020 •
edited
Loading

一点微小的测试结果：

  • subType 是否为 1 与注释中所谓“管理员邀请”并无关系。非管理员邀请加群、普通成员主动加群都有可能为 1
  • c2cInviteJoinGroupFlag 不能决定是否为邀请入群。
    • 当 c2cInviteJoinGroupFlag == 0 && groupMsgType == 2 时为管理员或普通成员邀请 bot 加群
    • 当 c2cInviteJoinGroupFlag == 0 && groupMsgType == 1 时为普通成员主动加群
    • 测试中未出现 c2cInviteJoinGroupFlag == 1 的情况

上述测试结果仅有一例样本，可能不准确。

@ryoii
Copy link
Member

ryoii commented Jun 28, 2020

好的，会作为已有数据的参考

@Him188 Him188 added the x:protocol 属性: 协议相关 label Jun 28, 2020
@Him188 Him188 added this to the 1.2.0 milestone Jul 3, 2020
@Him188 Him188 added the M 优先级: 主要 label Jul 3, 2020
@Him188 Him188 closed this in becc54d Jul 8, 2020
Him188 pushed a commit that referenced this pull request Dec 6, 2021
@Nambers
docs-register command (#402)
0864c92 
* 注册指令

这一节好像少了个注册指令

* Update Commands.md

* Update Commands.md
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
M 优先级: 主要 x:protocol 属性: 协议相关
Projects
None yet
Milestone
1.2.0
Development

Successfully merging this pull request may close these issues.
3 participants
@kenvix @ryoii @Him188