Skip to content

[fixed] hanging leaf node connection when account can't be found #2267

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
Merged
merged 3 commits into from
Jun 10, 2021
Merged

[fixed] hanging leaf node connection when account can't be found #2267

merged 3 commits into from
Jun 10, 2021

Conversation

matthiashanel
Copy link
Contributor

@matthiashanel matthiashanel commented Jun 8, 2021 •
edited
Loading

as a result of the issue, the leaf node connection never got created,
even after the account can be found.

Also tracing account id and name (when available)

Signed-off-by: Matthias Hanel [email protected]

[48700] 2021/06/08 15:02:46.327026 [INF] 127.0.0.1:4224 - lid:12 - Leafnode connection created for account: ACIB3OAWRMNK34DVSNYVQOEHH5RK7GNPHI7RPYZKVBX4UPD4262H2M2G/EXCACC

[48699] 2021/06/08 15:03:59.174597 [INF] 127.0.0.1:4224 - lid:10 - Leafnode connection closed: Server Shutdown account: ADXKBBHIU3HAY3VXAEETH3XSKQSFJ3YUH7NOBHGS5BN3K4SKNUNE7TUV/TEST
[48699] 2021/06/08 15:03:59.174824 [INF] 127.0.0.1:4224 - lid:11 - Leafnode connection closed: Server Shutdown account: ACIB3OAWRMNK34DVSNYVQOEHH5RK7GNPHI7RPYZKVBX4UPD4262H2M2G/EXCACC
[48699] 2021/06/08 15:03:59.175225 [INF] 127.0.0.1:4224 - lid:12 - Leafnode connection closed: Server Shutdown account: ABDGQHM46B4NBP4YIGKBDASU53TZHWPUVI33H5O3RQUPIGZ2QAFGJQKO/SYS

@matthiashanel
[fixed] hanging leaf node connection when account can't be found
9dac526 
as a result of the issue, the leaf node connection never got created,
even after the account can be found.

Also tracing account id and name (when available)

Signed-off-by: Matthias Hanel <[email protected]>
@matthiashanel matthiashanel requested a review from kozlovic June 8, 2021 19:03
kozlovic
kozlovic requested changes Jun 8, 2021
View reviewed changes
@matthiashanel
review comments
1a94046 
Signed-off-by: Matthias Hanel <[email protected]>
kozlovic
kozlovic requested changes Jun 8, 2021
View reviewed changes
@matthiashanel
changed close reason
2cca4f4 
Signed-off-by: Matthias Hanel <[email protected]>
kozlovic
kozlovic approved these changes Jun 8, 2021
View reviewed changes
Copy link
Member

@kozlovic kozlovic left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@matthiashanel matthiashanel merged commit 83389db into master Jun 10, 2021
@matthiashanel matthiashanel deleted the ln-conn-close branch June 10, 2021 15:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@kozlovic kozlovic kozlovic approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@matthiashanel @kozlovic