search

lora-aprs / LoRa_APRS_iGate

This is a LoRa APRS iGate/Digi based on an ESP32
https://www.lora-aprs.info
MIT License
357 stars 112 forks source link

Several fixes and watchdog #218

Closed mdiepart closed 2 years ago

mdiepart commented 2 years ago

Hi,

I would like to suggest several small changes to the firmware:

  1. Disable syslog by default. It seems weird to me that the logs are sent to an unkown server. You should let the user enable that for them if they want (I do not even know how this works ?)

  2. Add a watchdog timer in case the device hangs unexpectedly (Also addresses a current issue, #95 ).

  3. Fix the monitor flag in the plaformio.ini (otherwise the terminal will not display colored text)

  4. An extra "%s" was left in a log line (TaskRadiolib).

peterus commented 2 years ago

some answers to your questions:

  1. it looks like the the server crashed completely , it would be accessible here: http://syslog.lora-aprs.info/ because of this I want to remove the syslog feature asap in a new branch.
  2. thanks for the watchdog :)
  3. thanks! looks like this was changed in the last platformio releases
  4. thanks 👍
peterus commented 2 years ago

@mdiepart thanks for the fixes!

mdiepart commented 2 years ago

My pleasure! I will come back again if I have more suggestions!