Skip to content

Fix for FACILITY transfer#36

Open
exaSR wants to merge 1 commit into
graylog-labs:masterfrom
exaSR:master
Open

Fix for FACILITY transfer#36
exaSR wants to merge 1 commit into
graylog-labs:masterfrom
exaSR:master

Conversation

@exaSR

@exaSR exaSR commented Jan 28, 2021

Copy link
Copy Markdown

Fixes #35
I came upon the problem that the server part would not report the proper facility for messages.

This patch fixes the problem by

  • changing the constants in SyslogUtiliites to match RFC 5424 (this fixes the decoding part)
  • fixing the encoding accordingly

- changed message generation and parsing code accordingly
- added testcase
@CLAassistant

Copy link
Copy Markdown

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
You have signed the CLA already but the status is still pending? Let us recheck it.

@exaSR

exaSR commented Jan 28, 2021

Copy link
Copy Markdown
Author

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
You have signed the CLA already but the status is still pending? Let us recheck it.

Feel free to manually transfer my changes. I do not feel comfortable with granting @CLAassistant the requested permissions on my account...

@kodjo-anipah kodjo-anipah force-pushed the master branch 3 times, most recently from 333c694 to 3ea43cb Compare March 20, 2023 07:03
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.

Syslog Server fails to decode message facilities

2 participants