timmbogner / Farm-Data-Relay-System

A system that uses ESP-NOW, LoRa, and other protocols to transport sensor data in remote areas without relying on WiFi.
MIT License
507 stars 114 forks source link

Add MQTT port selection and MQTT user/password auth support #23

Closed thefeiter closed 2 years ago

thefeiter commented 2 years ago

First of all: Great Project! Got it up and running with spare parts laying around in under an hour.

In my setup, I want to use an MQTT broker on my server which requires authentication, so I added a few lines to be able to set MQTT user credentials. Also I added an option to select the port of the broker.

It works in my test setup.

Maybe it can be simplified.

timmbogner commented 2 years ago

Yo, thanks! I appreciate the help! The dev branch has some drastic changes happening, so I think that explains the conflicts? I will have a look on Monday and possibly just pull this to the main branch while the new version is under testing/construction.

Devilbinder commented 2 years ago

I unfortunately can't see what is causing the conflict. This code does not seem to affect the changes that I have made. Specifically left the current gateway examples alone until I have conformation that the Universal Gateway example is working.

Devilbinder commented 2 years ago

Resolved the conflict on https://github.com/Devilbinder/Farm-Data-Relay-System/tree/dev2 @thefeiter you should just need to merge my branch into yours to resolve the conflict.

thefeiter commented 2 years ago

Done, thanks!