fashberg / WThermostatBeca

Open Source firmware replacement for Tuya Wifi Thermostate from Beca and Moes with Home Assistant Autodiscovery
344 stars 71 forks source link

Mqtt port change #156

Closed dmitijmaslak closed 2 years ago

dmitijmaslak commented 2 years ago

Hello, please remove restriction of mqtt port...i have port 44444....but your firmware only 4 symbols.... Please fix this

fashberg commented 2 years ago

Hey, the base library WAdapter is saving port as string into the small flash, with fixed len to 4 chars. You are right, port range is up to 65565, but changing this means that the config layout has to be upgraded. Maybe sometimes. For now not changeable, except you fork. Easier to change port to something <=9999

Folke

dmitijmaslak commented 2 years ago

I have already removed this limitation and recompiled the firmware. thank you )

dieterlind commented 2 years ago

@dmitijmaslak : Can you give me a hint where you changed what to make 5 digits possible! Thanks!

dmitijmaslak commented 2 years ago

@dmitijmaslak : Can you give me a hint where you changed what to make 5 digits possible! Thanks!

Edit: lib\WAdapter\WNetwork.h Search “mqttport” and change: 4 to you need, and 1883 - if you need change default port use…