MoonModules / WLED

Control WS2812B and many more types of digital RGB LEDs with an ESP8266 or ESP32 over WiFi! MoonModules adds features on top of upstream.
https://mm.kno.wled.ge
GNU General Public License v3.0
185 stars 54 forks source link

Wireguard usermod folder not included #137

Open jameskennemore opened 2 weeks ago

jameskennemore commented 2 weeks ago

What happened?

The wireguard usermods directory doesn't exist, although wireguard is referenced in const.h. When re-adding the wireguard usermod directory and platformio_override.ini, Wireguard configuration options aren't in the Usermods config page after building. (It works in the non-MM build.)

To Reproduce Bug

Add the wireguard usermod directory and platformio_override.ini, then build. Wireguard configuration options aren't in the Usermods config page.

Expected Behavior

Wireguard config options should be in the Usermods config page after adding the wireguard usermod.

Install Method

Self-Compiled or other

What version/release of MM WLED?

MoonModules v0.14.1-b30

Which microcontroller/board are you seeing the problem on?

ESP32

Relevant log/trace output

No response

Anything else?

No response

Code of Conduct

jameskennemore commented 2 weeks ago

Added it back to the usermods_list.cpp and the configuration returned:

ifdef USERMOD_WIREGUARD

usermods.add(new WireguardUsermod());

endif

ifdef USERMOD_WIREGUARD

include "../usermods/wireguard/wireguard.h"

endif

softhack007 commented 2 weeks ago

Hi, I'm not sure this usermod would work in MM.

We don't have the changes from upstream that allow accessing UI pages from behind a proxy (https://github.com/Aircoookie/WLED/pull/3238). Not sure if wireguard relies on this feature (I don't have a setup for testing).