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

Implement support for structuredData option (SD-PARAM in RFC 5424). #110

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

Conversation

felix-hoc
Copy link

This addresses issue #106

@ScottChapman
Copy link

bump

@Schweigi
Copy link

Schweigi commented Mar 13, 2019

Would be great to have this PR merged. For example www.loggly.com requires structured data containing the API token to be passed when sending logs to their Syslog endpoint.

@cjbarth
Copy link
Contributor

cjbarth commented Oct 14, 2019

Should the README be updated to include directions about how to use this feature, and the fact that it is supported?

@felix-hoc
Copy link
Author

I've updated the Readme, let me know is this is enough or if I shoud add more info :) I kept it short for now since it is anyway explained in glossy.

@cjbarth
Copy link
Contributor

cjbarth commented Oct 28, 2019

This LGTM, but Travis doesn't like it.

@zenonhun
Copy link

What is the status of this change?

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.

5 participants