Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

fix(outputs.syslog): Trim field-names belonging to explicit SDIDs correctly #16014

Merged
merged 1 commit into from
Oct 15, 2024

Conversation

srebhan
Copy link
Member

@srebhan srebhan commented Oct 14, 2024

Summary

Trim explicit SD-IDs correctly from field names by using TrimPrefix instead of the inappropriate TrimLeft.

Checklist

  • No AI generated code was used in this PR

Related issues

resolves #16012

@telegraf-tiger telegraf-tiger bot added area/syslog fix pr to fix corresponding bug plugin/output 1. Request for new output plugins 2. Issues/PRs that are related to out plugins labels Oct 14, 2024
@srebhan srebhan force-pushed the syslog_issue_16012 branch from 39ae2ee to 275a9fd Compare October 14, 2024 16:56
@srebhan srebhan self-assigned this Oct 14, 2024
@srebhan srebhan added the ready for final review This pull request has been reviewed and/or tested by multiple users and is ready for a final review. label Oct 15, 2024
@srebhan srebhan assigned DStrand1 and unassigned srebhan Oct 15, 2024
@DStrand1 DStrand1 merged commit 0b1581c into influxdata:master Oct 15, 2024
31 checks passed
@github-actions github-actions bot added this to the v1.32.2 milestone Oct 15, 2024
asaharn pushed a commit to asaharn/telegraf that referenced this pull request Oct 16, 2024
srebhan added a commit that referenced this pull request Oct 28, 2024
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
area/syslog fix pr to fix corresponding bug plugin/output 1. Request for new output plugins 2. Issues/PRs that are related to out plugins ready for final review This pull request has been reviewed and/or tested by multiple users and is ready for a final review.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

syslog plugin mangles output
2 participants