Skip to content

Downgrade hivemq-client-mqtt to 1.3.7, netty 4.1.122 - #4976

Merged
wborn merged 2 commits into
openhab:mainfrom
holgerfriedrich:pr-netty-for-137
Aug 24, 2025
Merged

Downgrade hivemq-client-mqtt to 1.3.7, netty 4.1.122#4976
wborn merged 2 commits into
openhab:mainfrom
holgerfriedrich:pr-netty-for-137

Conversation

@holgerfriedrich

Copy link
Copy Markdown
Member

hivemq-client-mqtt 1.3.8 does lead to failing itests in add-ons repo (mqtt.homeassistant, mqtt.homie, mqtt.ruuvygateway), due to issue with transitive dependency dagger. I could not make it work until now, so I propose to revert the upgrade to 1.3.8 and stay with the also recently introduced 1.3.7.

This PR upgrades netty to 4.1.122 to match hivemq. This was missing in #4958.

Please do not squash during merge.

@holgerfriedrich
holgerfriedrich requested a review from a team as a code owner August 24, 2025 09:10
This reverts commit f23304e.

Signed-off-by: Holger Friedrich <mail@holger-friedrich.de>
Signed-off-by: Holger Friedrich <mail@holger-friedrich.de>
@holgerfriedrich

Copy link
Copy Markdown
Member Author

Finally build of core and add-ons completed, itests are fine again.
https://github.com/holgerfriedrich/openhab-builds/actions/runs/17186380628

@wborn wborn left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Thanks!

@wborn
wborn merged commit f7e07bd into openhab:main Aug 24, 2025
4 checks passed
@wborn wborn added this to the 5.1 milestone Aug 24, 2025
@holgerfriedrich
holgerfriedrich deleted the pr-netty-for-137 branch August 24, 2025 11:26
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