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

Fix crash when QoS value is 2 #1

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

Conversation

kenji21
Copy link

@kenji21 kenji21 commented Apr 30, 2020

fix for this issue: aciidgh#54

kenji21 and others added 2 commits April 30, 2020 08:20
When a publish was received with QoS of 0 there was a publish Acknowledgment sent with a MessageID of 0 causing the server to send an endEncountered event closing the socket.
@kenji21
Copy link
Author

kenji21 commented Apr 30, 2020

added a commit johnsonajason@8dbe617 from another fork that fix issue aciidgh#46 (which correspond to PR aciidgh#43)

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