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

Deal with optional newline in event #2

Merged
merged 1 commit into from
Mar 18, 2025
Merged

Deal with optional newline in event #2

merged 1 commit into from
Mar 18, 2025

Conversation

widhalmt
Copy link
Member

After ingesting a few more events we found out there are different variants on the system:

  • single line events that worked just as we have them in the pattern so far
  • multline events where the timestamp is the first line and then everything is in the next line, starting at the begin of the line
  • mutline events where all but the first line with the timestamp are indented

This new pattern should be able to deal with all of them.

@widhalmt widhalmt added the bug Something isn't working label Mar 18, 2025
@widhalmt widhalmt requested a review from afeefghannam89 March 18, 2025 13:03
@widhalmt widhalmt self-assigned this Mar 18, 2025
Copy link
Member

@afeefghannam89 afeefghannam89 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks

@widhalmt widhalmt merged commit dc0630f into main Mar 18, 2025
2 checks passed
@widhalmt widhalmt deleted the fix/newline branch March 18, 2025 13:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants