Skip to content

fix for bug causing a crash twice every hour, at minutes :08 and :09#65

Merged
Ym0T merged 1 commit intoYm0T:mainfrom
forestfire2397:Cron-Engine
Apr 17, 2026
Merged

fix for bug causing a crash twice every hour, at minutes :08 and :09#65
Ym0T merged 1 commit intoYm0T:mainfrom
forestfire2397:Cron-Engine

Conversation

@forestfire2397
Copy link
Copy Markdown
Contributor

The crash happens twice every hour, at minutes :08 and :09. Bash treats numbers with leading zeros as octal — 08 and 09 are invalid octal digits, so $((60 - 09)) throws a fatal error and kills the whole cron engine daemon.

@Ym0T Ym0T self-requested a review April 17, 2026 18:45
@Ym0T
Copy link
Copy Markdown
Owner

Ym0T commented Apr 17, 2026

Thanks for catching this! Great fix, merging it in 🙌

@forestfire2397
Copy link
Copy Markdown
Contributor Author

Thanks for catching this! Great fix, merging it in 🙌

no problem, a bit new to github and how to push changes and stuff lol. but this was a bug i fix a while ago as i was running a job every minute and it would "randomly crash".
i thought i was merging to my fork, i was going to do a few things before requesting from the origin lol

@Ym0T Ym0T merged commit da308f6 into Ym0T:main Apr 17, 2026
4 checks passed
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