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

set state to logon before return #12

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

Conversation

Leeeft
Copy link
Collaborator

@Leeeft Leeeft commented Feb 7, 2024

问题:
return 前不将 session 状态变更为 logon,当后续逻辑发生 error return 时,会让 session 一直处于 latent 状态,并且不会再次发起连接。
解决方式:
当 fix session 处于 latent 状态,并且准备发起 Connect 时,说明 TCP 连接已经建立,进行后续可能 error return 的逻辑之前,先将 session 状态变更为 logon,借由 logon 超时的回调,解决上述问题。并且此时 TCP 已经建立连接,变更为 logon 有其业务合理性。

@Leeeft
Copy link
Collaborator Author

Leeeft commented Feb 7, 2024

Snipaste_2024-02-07_18-12-32

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant