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.

Sign up for GitHub

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

Jump to bottom

Bug: logon-summary crash #1477

Closed
YamatoSecurity opened this issue Nov 7, 2024 · 1 comment · Fixed by #1476
Closed

Bug: logon-summary crash #1477

YamatoSecurity opened this issue Nov 7, 2024 · 1 comment · Fixed by #1476
Assignees
@fukusuket
Labels
bug Something isn't working
Milestone
2.19.0

Comments

@YamatoSecurity
Copy link
Collaborator

When there is a corrupted log, logon-summary will crash:

[ERROR] timestamp parse error. input: null input contains invalid characters
thread 'main' panicked at src/timeline/metrics.rs:230:36:
called `Option::unwrap()` on a `None` value
@YamatoSecurity YamatoSecurity added the bug Something isn't working label Nov 7, 2024
@YamatoSecurity YamatoSecurity added this to the 2.19.0 milestone Nov 7, 2024
@YamatoSecurity YamatoSecurity mentioned this issue Nov 7, 2024
Merged
@YamatoSecurity
Copy link
Collaborator Author

I noticed that this only happens with -x is used and therefore the events are more corrupt than normal.
This may be related to #1418
Someone created a PR to fix something but then closed it and never re-opened it so we might also want to fix this bug as well.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@fukusuket fukusuket

Labels
bug Something isn't working
Projects
None yet
Milestone
2.19.0
Development

Successfully merging a pull request may close this issue.

feat: add RDS log to logon-summary Yamato-Security/hayabusa
2 participants
@fukusuket @YamatoSecurity