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

network(), syslog() sources: add $PEERIP and $PEERPORT macros #5291

Open
wants to merge 5 commits into
base: develop
Choose a base branch
from

Conversation

HofiOne
Copy link
Collaborator

@HofiOne HofiOne commented Mar 21, 2025

The $PEERIP and $PEERPORT macros always display the address and port of the direct sender.
In most cases, these values are identical to $SOURCEIP and $SOURCEPORT.
However, when dealing with proxied protocols, $PEERIP and $PEERPORT reflect the proxy's address and port,
while $SOURCEIP and $SOURCEPORT indicate the original source of the message.

Backport of 523 by @MrAnno

MrAnno added 3 commits March 21, 2025 15:14
Signed-off-by: László Várady <laszlo.varady@anno.io>
Signed-off-by: László Várady <laszlo.varady@anno.io>
Signed-off-by: Hofi <hofione@gmail.com>
For example, to store the address of the proxy.

Signed-off-by: László Várady <laszlo.varady@anno.io>
Signed-off-by: Hofi <hofione@gmail.com>
@HofiOne HofiOne force-pushed the backport/proxy-addr branch from e179d2f to f25340d Compare March 21, 2025 14:41
@HofiOne HofiOne changed the title Backport/proxy addr network(), syslog() sources: add $PEERIP and $PEERPORT macros Mar 21, 2025
@kira-syslogng
Copy link
Contributor

Build FAILURE

MrAnno added 2 commits March 22, 2025 17:29
Signed-off-by: László Várady <laszlo.varady@anno.io>
Signed-off-by: Hofi <hofione@gmail.com>
Signed-off-by: László Várady <laszlo.varady@anno.io>
Signed-off-by: Hofi <hofione@gmail.com>
@HofiOne HofiOne force-pushed the backport/proxy-addr branch from f25340d to b4a4cb9 Compare March 22, 2025 16:29
@HofiOne HofiOne marked this pull request as ready for review March 24, 2025 09:27
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.

4 participants