Skip to content
This repository has been archived by the owner on Feb 5, 2024. It is now read-only.

Detect and handle when MQTT URI hasn't been set #81

Open
vringar opened this issue Aug 14, 2022 · 0 comments
Open

Detect and handle when MQTT URI hasn't been set #81

vringar opened this issue Aug 14, 2022 · 0 comments
Labels
enhancement New feature or request MatterHub Concerns our ESP32

Comments

@vringar
Copy link
Contributor

vringar commented Aug 14, 2022

Currently if the user forgot to setup their config properly, matterHub will just read the empty string and crash:

I (1245) MQTT: This is my name:  and this is my key
I (1245) MQTT: [APP] Free memory: 130232 bytes
Guru Meditation Error: Core  0 panic'ed (LoadProhibited). Exception was unhandled.

IDK if we can use __static_assert or something else in the build to detect that this is unset and refuse to build.
Or we crash at runtime with a descriptive error message.

@vringar vringar added enhancement New feature or request MatterHub Concerns our ESP32 labels Aug 14, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
enhancement New feature or request MatterHub Concerns our ESP32
Projects
None yet
Development

No branches or pull requests

1 participant