Skip to content

Fix issue 3501 #10374

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

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

Fix issue 3501 #10374

wants to merge 3 commits into from

Conversation

chiese-s
Copy link

Title: Enhance syslog input plugin with raw message and source address support

Description:
This PR enhances the syslog input plugin by adding support for:

  1. Including raw syslog messages in the output records via a configurable key
  2. Adding source IP address information to records via a configurable key

Key changes:

  • Added append_message_to_record_data() helper function to handle message expansion
  • Modified pack_line() to optionally include raw message and source address
  • Improved error handling and logging for message processing

The changes maintain backward compatibility while providing more flexibility in syslog record processing.

@patrick-stephens
Copy link
Contributor

Please follow the contribution guidelines around PR titles and other requirements like the DCO check.

@patrick-stephens
Copy link
Contributor

Issue 3501 seems completely unrelated to me: #3501

Copy link
Contributor

@patrick-stephens patrick-stephens left a comment

Choose a reason for hiding this comment

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

The changes seem to just be logging related extra warning or debug messages and not what is described in the PR. Is this correct?

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

Successfully merging this pull request may close these issues.

2 participants