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

Sane defaults for remote syslog #16300

Merged
merged 1 commit into from
Jan 23, 2023
Merged

Sane defaults for remote syslog #16300

merged 1 commit into from
Jan 23, 2023

Conversation

mtdcr
Copy link
Contributor

@mtdcr mtdcr commented Jan 22, 2023

The most commonly used transport for remote syslog is port 514/udp. It also is the only defined method for BSD syslog (RFC 3164). RFCs 5424+5425 specify an additional TLS transport via 6514/tcp, which does not seem to be supported by zigbee2mqtt, as evidenced by the lack of TLS-related configuration options in this area. RFC 6587 describes legacy uses of syslog over TCP, recommending against them.

Port 123, the previous default, was probably used by accident and is reserved for NTP.

$ grep -w -e 123 -e 514 -e 601 -e 6514 /etc/services
ntp	123/udp				# Network Time Protocol
shell	514/tcp		cmd syslog	# no passwords used
syslog	514/udp
syslog-tls	6514/tcp		# Syslog over TLS [RFC5425]

The most commonly used transport for remote syslog is port 514/udp.
It also is the only defined method for BSD syslog (RFC 3164). RFCs
5424+5425 specify an additional TLS transport via 6514/tcp, which
does not seem to be supported by zigbee2mqtt, as evidenced by the
lack of TLS-related configuration options in this area. RFC 6587
describes legacy uses of syslog over TCP, recommending against them.

Port 123, the previous default, was probably used by accident and
is reserved for NTP.

| $ grep -w -e 123 -e 514 -e 601 -e 6514 /etc/services
| ntp		123/udp				# Network Time Protocol
| shell		514/tcp		cmd syslog	# no passwords used
| syslog	514/udp
| syslog-tls	6514/tcp			# Syslog over TLS [RFC5425]
Koenkk added a commit to Koenkk/zigbee2mqtt.io that referenced this pull request Jan 23, 2023
@Koenkk Koenkk merged commit 06820d2 into Koenkk:dev Jan 23, 2023
@Koenkk
Copy link
Owner

Koenkk commented Jan 23, 2023

Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants