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

Push Connect Update #4

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

Push Connect Update #4

wants to merge 3 commits into from

Conversation

W192547975
Copy link

Update pushhandler.go

@W192547975
PushRsp Debug
fdc6706 
Update pushhandler.go
@W192547975
Copy link
Author

W192547975 commented May 18, 2024 •
edited
Loading

纠正了回复MsgPushRsp时缺少PushHeader的问题。

@W192547975 W192547975 closed this May 18, 2024
@W192547975 W192547975 reopened this May 18, 2024
@W192547975
Copy link
Author

抱歉，客户端源代码误导了我。似乎客户端的接收并不会出现明显的问题（虽然说会有一个不正确的binary解析）。我会重新考虑我的实现。

@W192547975 W192547975 closed this May 18, 2024
@W192547975
Copy link
Author

在完成之后我会重新打开这个pr

@W192547975
Revert "PushRsp Debug"
7c3fcca 
This reverts commit fdc6706.
@W192547975
Push Connect Update
c0ce330 
Update pushhandler.go
@W192547975

This comment was marked as duplicate.

Sign in to view
1 similar comment
@W192547975
Copy link
Author

修改已完成，但这是一个重大修改（算是吧）

@W192547975 W192547975 changed the title PushRsp Debug Push Connect Update May 25, 2024
@W192547975
Copy link
Author

嘶，刚才等墙的时候竟然发了两遍消息……

我取消了对PushConnectReq的TOTP验证，将pushPermission转为由匹配的PushConnectRsp错误码建立。

@W192547975 W192547975 reopened this May 25, 2024
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
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@W192547975