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

Bug with controlling_process for Connection #170

Merged
merged 1 commit into from
Jan 24, 2023

Conversation

klacke
Copy link
Contributor

@klacke klacke commented Jan 20, 2023

To reproduce, run

cd  test/example
make all
lux p_connect.lux

@qzhuyan
Copy link
Collaborator

qzhuyan commented Jan 20, 2023

relates to #169 ?

@qzhuyan
Copy link
Collaborator

qzhuyan commented Jan 20, 2023

I got this:\

- Stream
+  {stream, 1}.
+ Failed to start_stream {error,stm_open_error,invalid_state}
+ --> 

Is this the same error you get? I just want to make sure we are checking the same trouble.

@klacke
Copy link
Contributor Author

klacke commented Jan 20, 2023

Yes, same error, and no, not related AFAIK to #169

timer:sleep(400),
Self ! {connection, Conn};
Err ->
Self ! {connection_err, Err}
Copy link
Collaborator

Choose a reason for hiding this comment

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

looks like a monitoring issue,
add
timer:sleep(1000) solved the issue.

The connection is closed because the old owner is dead. I will check if it demon the old owner pid while swap the new owner.

Copy link
Collaborator

Choose a reason for hiding this comment

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

solution: #172

@qzhuyan qzhuyan merged commit f99c2d7 into emqx:main Jan 24, 2023
@qzhuyan
Copy link
Collaborator

qzhuyan commented Jan 24, 2023

test pass after merge.

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.

2 participants