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

Bumping mqtt dependency #61

Merged
merged 1 commit into from
May 30, 2021
Merged

Bumping mqtt dependency #61

merged 1 commit into from
May 30, 2021

Conversation

michael-robbins
Copy link
Contributor

@michael-robbins michael-robbins commented May 29, 2021

Testing out bumping the mqtt dependency to try and fix #60

@michael-robbins
Copy link
Contributor Author

Docker image builds, I'm currently running it on my servers, and it seems to at least 'still work', so that's a good thing, will run it for a few days and see if I experience any random disconnects

@@ -105,6 +107,8 @@ github.com/gorilla/context v1.1.1/go.mod h1:kBGZzfjB9CEq2AlWe17Uuf7NDRt0dE0s8S51
github.com/gorilla/mux v1.6.2/go.mod h1:1lud6UwP+6orDFRuTfBEV8e9/aOM/c4fVVCaMa2zaAs=
github.com/gorilla/mux v1.7.3/go.mod h1:1lud6UwP+6orDFRuTfBEV8e9/aOM/c4fVVCaMa2zaAs=
github.com/gorilla/websocket v0.0.0-20170926233335-4201258b820c/go.mod h1:E7qHFY5m1UJ88s3WnNqhKjPHQ0heANvMoAMk2YaljkQ=
github.com/gorilla/websocket v1.4.2 h1:+/TMaTYc4QFitKJxsQ7Yye35DkWvkdLcvGKqM+x0Ufc=
github.com/gorilla/websocket v1.4.2/go.mod h1:YR8l580nyteQvAITg2hZ9XVh4b55+EU/adAjf1fMHhE=
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Was brought in as a dependency from eclipse/paho.mqtt.golang, can verify this by checking their repo: https://github.com/eclipse/paho.mqtt.golang/blob/master/go.mod

@michael-robbins michael-robbins changed the title [WIP] Bumping mqtt dependency Bumping mqtt dependency May 30, 2021
@michael-robbins
Copy link
Contributor Author

Should be good to review and test @hikhvar, working on my setup at least, I saw it reconnect overnight at 2am by itself and later in the day yesterday.

And it successfully started receiving device messages this morning when my devices started messaging again.

@hikhvar hikhvar merged commit 6b5b3d6 into hikhvar:master May 30, 2021
@hikhvar
Copy link
Owner

hikhvar commented May 30, 2021

Thanks for the debugging and the PR. I will fix the CI later and create a Bugfix release.

@hikhvar
Copy link
Owner

hikhvar commented May 30, 2021

Just released v0.1.6-RC8.

I need some docs rework before releasing v0.1.6 finally.

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.

MQTT listener goes 'stale'
2 participants