[Bug]: SimpleJSON is unmaintained
We use code from the SimpleJSON library: https://github.com/MJPA/SimpleJSON Specifically,
- src/serialization/JSON{.cpp,h}
- src/serialization/JSONValue{.cpp,h}
This code has not been updated in 8 years and the project appears unmaintained.
We should consider courses of action such as:
- Investigate potential alternate libraries
- Determine whether the meshtastic community should take on a maintainer role for this code
- Patch some of the identified flaws
This is already the 2nd JSON lib we are using. Obvious choice would be the well maintained ArduinoJSON but is is rather big for the task.
https://github.com/meshtastic/json11
Suggestions welcome.
JSON should leave the device and be managed on a PI or something with more resources.
I vote for adding an MQTT bridge to the Meshtastic portfolio, and removing the JSON feature from the firmware.
Probably wouldn't be hard to extend https://github.com/fifieldt/meshtastic-map-backend
to bridge MQTT and JSON in general, rather than just for maps. Could maybe even add some native homeassistant json stuff to make that integration easier...
@pdxlocations also has some nice MQTT bridge code.
We probably need to accelerate this being replaced.
This is already the 2nd JSON lib we are using. Obvious choice would be the well maintained ArduinoJSON but is is rather big for the task.
https://github.com/meshtastic/json11
Suggestions welcome.
This is also abandoned by dropbox (maybe even older than our current library)
this is what it looks like with ArduinoJson: https://github.com/meshtastic/firmware/pull/4661
Has anyone looked into the protobuf library for this conversion? https://github.com/protocolbuffers/protobuf/blob/main/src/google/protobuf/util/json_util.h
JSON functionality in the firmware is in the process of being retired https://github.com/meshtastic/firmware/issues/5507