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

fix: update go-c8y lib to fix notifications2 connect/reconnect errors #407

Merged

reubenmiller merged 3 commits into v2 from fix-notifications2-connect-errors

Oct 2, 2024

Owner

reubenmiller commented Oct 2, 2024 •

edited

Loading

Fixes to the c8y notification2 commands regarding the websocket connection to Cumulocity IoT and the parsing of the messages.

Changes include:

Fix message parsing bug which was caused by failing to allocate memory during parsing which resulted in corrupted message parsing (fields were mixed). This would result in corrupted ack messages which the server would then disconnect the client, and a reconnection loop would start.
Update websocket library to prevent use of closed network connection errors
Explicitly close websocket before reconnecting

reubenmiller added 3 commits

October 2, 2024 12:47


          fix: update go-c8y lib to fix notifications2 connect/reconnect errors

8d7e044


          include notification2 message id in the info log message

cd20350


          fix: update go-c8y lib to fix notifications2 message parsing bug

e2334e3

reubenmiller merged commit 200b418 into v2

4 checks passed

reubenmiller deleted the fix-notifications2-connect-errors branch

October 2, 2024 13:45

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet