Skip to content
This repository has been archived by the owner on May 21, 2024. It is now read-only.

RFC3164: parsing syslog without PRI part #30

Open
dimonomid opened this issue Mar 1, 2020 · 2 comments
Open

RFC3164: parsing syslog without PRI part #30

dimonomid opened this issue Mar 1, 2020 · 2 comments
Labels
enhancement New feature or request help wanted Extra attention is needed

Comments

@dimonomid
Copy link

When syslog is being saved to files, the PRI value is typically removed (I don't know real reason for that though, but it's just my observation). And go-syslog complains that it can't find a PRI value when parsing syslog files. I worked it around by prepending every line with some dummy priority before parsing, but could we add at least an option to support syslog messages without a PRI value?

Also, as of now the Parse method just takes a byte array, there is no option to parse from an io.Reader; it makes prepending messages expensive. Can we make it possible to parse from an io.Reader as well? (It probably warrants for a separate issue, but it's more of a related question for now, so leaving it here)

Thanks!

@russorat russorat added enhancement New feature or request help wanted Extra attention is needed labels Mar 2, 2020
@russorat
Copy link

russorat commented Mar 2, 2020

@dimonomid thanks for the suggestions. Those sound like great additions. We will consider it for a future release. Would you be willing to help by contributing to the library?

@goller goller removed their assignment Apr 13, 2020
@haimrubinstein
Copy link

@russorat I know I'm a bit late :)
I would love to contribute to this.
I opened an issue on it
#52

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
enhancement New feature or request help wanted Extra attention is needed
Projects
None yet
Development

No branches or pull requests

4 participants