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

MqDockerUp stopped due to an error #68

Closed
chriswood1001 opened this issue May 25, 2023 · 5 comments · Fixed by #74
Closed

MqDockerUp stopped due to an error #68

chriswood1001 opened this issue May 25, 2023 · 5 comments · Fixed by #74
Labels
issue:bug Something isn't working

Comments

@chriswood1001
Copy link

The container appears to be in an endless restart approximately every minute.

I'm running V0.16.2.

2023-05-25T02:30:31.506732761Z 25.05.2023 02:30:31 [info] 	Finished checking for image updates
2023-05-25T02:30:31.507198747Z 25.05.2023 02:30:31 [info] 	Next check in 5 minutes
2023-05-25T02:30:36.969450468Z 25.05.2023 02:30:36 [error] 	MqDockerUp stopped due to an error
2023-05-25T02:30:36.969616858Z 25.05.2023 02:30:36 [error] 	object
2023-05-25T02:30:36.977221950Z 25.05.2023 02:30:36 [error] 	SyntaxError: Unexpected end of JSON input
2023-05-25T02:30:36.977240122Z     at JSON.parse (<anonymous>)
2023-05-25T02:30:36.977243174Z     at /app/src/index.ts:48:21
2023-05-25T02:30:36.977245454Z     at Generator.next (<anonymous>)
2023-05-25T02:30:36.977247740Z     at /app/dist/src/index.js:8:71
2023-05-25T02:30:36.977249681Z     at new Promise (<anonymous>)
2023-05-25T02:30:36.977251596Z     at __awaiter (/app/dist/src/index.js:4:12)
2023-05-25T02:30:36.977253489Z     at MqttClient.<anonymous> (/app/src/index.ts:47:60)
2023-05-25T02:30:36.977255445Z     at MqttClient.emit (node:events:511:28)
2023-05-25T02:30:36.977257358Z     at MqttClient._handlePublish (/app/node_modules/mqtt/lib/client.js:1547:12)
2023-05-25T02:30:36.977259409Z     at MqttClient._handlePacket (/app/node_modules/mqtt/lib/client.js:535:12)
2023-05-25T02:30:37.916465879Z 
2023-05-25T02:30:37.916490301Z > mqdockerup@0.16.2 start
2023-05-25T02:30:37.916494804Z > node dist/src/index.js
@MichelFR
Copy link
Owner

That's because there are some messages being sent in MQTT which are not formatted as JSON.

I'll make it compatible to that by ignoring those values, that should fix it.

I'm gonna use this issue to track the progress. Thanks for bringing this to my attention.

@MichelFR MichelFR added the issue:bug Something isn't working label May 26, 2023
@knorrre
Copy link

knorrre commented Jun 6, 2023

Same problem here. Is there a solution yet?

MichelFR added a commit that referenced this issue Jun 8, 2023
@MichelFR
Copy link
Owner

MichelFR commented Jun 8, 2023

Should be fixed in next release

@MichelFR MichelFR linked a pull request Jun 8, 2023 that will close this issue
MichelFR added a commit that referenced this issue Jun 8, 2023
@chriswood1001
Copy link
Author

The error is now reporting as a graceful warning. Critical issue resolved.

Is there a way to determine what JSON input is failing as my logs are flooded with this now. It's unclear what's needed to rectify.

@MichelFR
Copy link
Owner

The only thing you could do is to subscribe to the MQTT yourself and check which data is beeing sent there.
In Future im gonna rewrite some logic, so that i wont listen to it at all.

MichelFR added a commit that referenced this issue Mar 4, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
issue:bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants