-
-
Notifications
You must be signed in to change notification settings - Fork 511
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add syslog logger implementation #2292
base: master
Are you sure you want to change the base?
Conversation
@khschmidt in #2278 has successfully built and used this PR to configure his OpenDTU using remote syslog. @tbnobody would you want to merge this into master or are there some objections from the project side ? |
This implements RFC5424 version of the protocol. Doesn't use https://github.com/arcao/Syslog since the protocol itself is trivial and most of the libraries functionality is not needed here. The library also doesn't support setting the PROCID field, which is set to a random id to indicate a reboot here. Adds UI for syslog configuration to network admin view.
* avoid duplicate id for hostname input * hide server and port inputs if syslog disabled
Updated to match the version that was committed into OpenDTU-OnBattery: |
I am running this patch since 2 weeks now and OpenDTU creates 20MB logs per day (three inverters). |
Adding ESP-IDF Logging framework might be a way to filter log file generation in Serial, Web Console and Syslog #2361 |
This implements RFC5424 version of the protocol.
See #1819
I didn't use https://github.com/arcao/Syslog since the protocol itself is trivial and most of the libraries functionality is not needed here. The library also doesn't support setting the PROCID field, which is set to a random id to indicate a reboot here.