Skip to content
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 #2280

Closed
wants to merge 2 commits into from
Closed

Conversation

ranma
Copy link
Contributor

@ranma ranma commented Sep 15, 2024

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.

@ranma ranma force-pushed the tnobody-syslog branch 3 times, most recently from a429c31 to 6a1f1eb Compare September 15, 2024 10:39
This implements RFC5424 version of the protocol.

See tbnobody#1819
@ranma
Copy link
Contributor Author

ranma commented Sep 15, 2024

2024-09-15T12:41:21.466816+02:00 OpenDTU-347AF0 OpenDTU[e4975f6a] [SyslogLogger] Logging to 192.168.8.240
2024-09-15T12:41:21.466816+02:00 OpenDTU-347AF0 OpenDTU[e4975f6a] WiFi got ip: 192.168.8.242
2024-09-15T12:41:21.474350+02:00 OpenDTU-347AF0 OpenDTU[e4975f6a] Network connected
2024-09-15T12:41:26.618582+02:00 OpenDTU-347AF0 OpenDTU[e4975f6a] Websocket: [/livedata][1] connect
2024-09-15T12:41:30.963775+02:00 OpenDTU-347AF0 OpenDTU[e4975f6a] Admin AP remaining seconds: 10 / 180
2024-09-15T12:41:40.977250+02:00 OpenDTU-347AF0 OpenDTU[e4975f6a] Admin AP remaining seconds: 20 / 180
2024-09-15T12:41:50.985588+02:00 OpenDTU-347AF0 OpenDTU[e4975f6a] Admin AP remaining seconds: 30 / 180
2024-09-15T12:42:00.999822+02:00 OpenDTU-347AF0 OpenDTU[e4975f6a] Admin AP remaining seconds: 40 / 180
2024-09-15T12:42:11.007716+02:00 OpenDTU-347AF0 OpenDTU[e4975f6a] Admin AP remaining seconds: 50 / 180
2024-09-15T12:42:11.575805+02:00 OpenDTU-347AF0 OpenDTU[e4975f6a] Websocket: [/livedata][1] disconnect
[...restarting from web UI, TASKID changes to indicate restart...]
2024-09-15T12:42:18.684665+02:00 OpenDTU-347AF0 OpenDTU[306b82ba] [SyslogLogger] Logging to 192.168.8.240
2024-09-15T12:42:18.684963+02:00 OpenDTU-347AF0 OpenDTU[306b82ba] WiFi got ip: 192.168.8.242
2024-09-15T12:42:18.685450+02:00 OpenDTU-347AF0 OpenDTU[306b82ba] Network connected
2024-09-15T12:42:28.188873+02:00 OpenDTU-347AF0 OpenDTU[306b82ba] Admin AP remaining seconds: 10 / 180
2024-09-15T12:42:38.199718+02:00 OpenDTU-347AF0 OpenDTU[306b82ba] Admin AP remaining seconds: 20 / 180
2024-09-15T12:42:48.210778+02:00 OpenDTU-347AF0 OpenDTU[306b82ba] Admin AP remaining seconds: 30 / 180
2024-09-15T12:42:58.221810+02:00 OpenDTU-347AF0 OpenDTU[306b82ba] Admin AP remaining seconds: 40 / 180

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is the hostname missing here ?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hostname re-uses the existing hostname locale entries which are already used in the WiFi configuration section.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks I like this +1

@ranma
Copy link
Contributor Author

ranma commented Sep 15, 2024

english UI

german UI

@ranma
Copy link
Contributor Author

ranma commented Sep 21, 2024

Hrm, hab' den Branch umbenannt (tnobody -> tbnobody), das hat natürlich heimlich, still und leise den PR geschlossen...

@ranma
Copy link
Contributor Author

ranma commented Sep 21, 2024

Re-opened as #2292

Copy link

This pull request has been automatically locked since there has not been any recent activity after it was closed. Please open a new discussion or issue for related concerns.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Oct 22, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants