Skip to content
This repository has been archived by the owner on Nov 17, 2020. It is now read-only.

amqp10_client_connection: Handle heartbeat in close_sent/2 #44

Merged
merged 1 commit into from
Feb 12, 2020

Conversation

dumbbell
Copy link
Member

This message, sent by the timer module, may happen in the close_sent state. We can ignore it but we still need to handle it.

I saw this case happen in CI:

*** System report during system_SUITE:init_per_group/2 for activemq 2020-02-12 06:06:06.473 ***
=WARNING REPORT==== 12-Feb-2020::06:06:06.473178 ===
terminating connection with reason '{function_clause,
                                     [{amqp10_client_connection,close_sent,
                                       [heartbeat,
                                        {state,2,<0.455.0>,
(...)

This message, sent by the timer module, may happen in the `close_sent`
state. We can ignore it but we still need to handle it.

I saw this case happen in CI:

*** System report during system_SUITE:init_per_group/2 for activemq 2020-02-12 06:06:06.473 ***
=WARNING REPORT==== 12-Feb-2020::06:06:06.473178 ===
terminating connection with reason '{function_clause,
                                     [{amqp10_client_connection,close_sent,
                                       [heartbeat,
                                        {state,2,<0.455.0>,
(...)
Copy link
Member

@michaelklishin michaelklishin left a comment

Choose a reason for hiding this comment

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

Apparently keep_state and keep_state_and_data are specific to gen_statem, so this looks reasonable.

@michaelklishin michaelklishin merged commit b4d3ce7 into master Feb 12, 2020
@michaelklishin michaelklishin deleted the handle-heartbeat-in-close_sent branch February 12, 2020 14:51
michaelklishin added a commit that referenced this pull request Feb 12, 2020
amqp10_client_connection: Handle heartbeat in close_sent/2

(cherry picked from commit b4d3ce7)
michaelklishin added a commit that referenced this pull request Apr 30, 2020
amqp10_client_connection: Handle heartbeat in close_sent/2

(cherry picked from commit b4d3ce7)
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants