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

append messages to log #1757

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Conversation

drexlma
Copy link

@drexlma drexlma commented Feb 16, 2025

not override the log

not override the log
@t3chn0m4g3
Copy link
Member

Thanks for the PR. However overwriting the log is intentional as it will be rotated by logrotate before being overwritten.

@drexlma
Copy link
Author

drexlma commented Feb 16, 2025

and if the container crashes? Or is the PC restarted? Or is Docker updated?

then the log is emptied.

@t3chn0m4g3
Copy link
Member

Yes, you are correct, those cases are not covered. At that point, the data would have been ingested into ES already.
For my better understanding (and not to annoy you, by any means 😅), are the logs outside of ES of greater value? If yes, I am happy to merge and also will look into updating logging for the other honeypots accordingly (if not already implemented).

Copy link

This pull request has been marked as stale because it has had no activity for 7 days. If you are still working on this, please comment or it will be closed in 7 days.

@github-actions github-actions bot added the stale label Feb 25, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants