-
Notifications
You must be signed in to change notification settings - Fork 3.9k
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
Add support for syslog server hostname #1658
Conversation
schlagert/syslog version 3.4.3 adds support for specifying a host name for `dest_host`, this PR supports it as well See this gist for my test procedure: git@gist.github.com:7ab6348839c3dc7b18b6662f024e089c.git
priv/schema/rabbit.schema
Outdated
@@ -1086,9 +1086,8 @@ end}. | |||
{datatype, {enum, [true, false]}} | |||
]}. | |||
|
|||
{mapping, "log.syslog.ip", "syslog.dest_host", [ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can we leave log.syslog.ip
in place for compatibility? (it also makes some sense as an alias anyway)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I wanted to do that, but since they both set syslog.dest_host
, it doesn't work.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Oh, OK.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hm there may be options if the value can be transformed, maybe. Let me check.
There are Cuttlefish limitations I wasn't aware of.
@michaelklishin - ready for re-review, both |
Thanks for the review @michaelklishin |
schlagert/syslog version 3.4.3 adds support for specifying a host name for
dest_host
, this PR supports it as wellSee this gist for my test procedure:
https://gist.github.com/lukebakken/20bd05816a8b80839cc3c6645350bfa4