Skip to content

Conversation

@Martinski4GitHub
Copy link
Collaborator

Added code to save the most recent F/W update log file. This way at least the most recent remains available to review when deleting all log files older than 30 days.

Added code to save the most recent F/W update log file. This way at least the most recent remains available to review when deleting all log files older than 30 days.
@ExtremeFiretop ExtremeFiretop merged commit 806f1e1 into ExtremeFiretop:dev Feb 8, 2026
1 check passed
@ExtremeFiretop
Copy link
Owner

Smart call, realistically I expect that most of the time, it would keep at least 30 days of daily logs.
However, it's possible the script is set to run manually, or monthly by cron job, in which case the "delete anything older than 30 days" would delete everything.

@Martinski4GitHub
Copy link
Collaborator Author

Smart call, realistically I expect that most of the time, it would keep at least 30 days of daily logs. However, it's possible the script is set to run manually, or monthly by cron job, in which case the "delete anything older than 30 days" would delete everything.

Yes, exactly. For the vast majority of users, the change would not make any difference, but my friend pointed out that he had no logs at all after setting up the cron job to run only once a month. (Why? Long story. Short version: he didn't want to update the F/W right away, so he set up a 60-day postponement with a once-a-month cron job).

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.

2 participants