AS1024/itflow
1
0
Fork 0
mirror of https://github.com/itflow-org/itflow synced 2026-08-05 07:07:14 +00:00
Code Issues Packages Projects Releases Wiki Activity

Scope the cron lock per install so instances don't block each other

Browse Source 
Report an unopenable cron lock file instead of claiming cron is running
This commit is contained in:
johnnyq
2026-07-27 16:41:21 -04:00
parent cff416dd87
commit 409d769eb7
1 changed files with 9 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files

13
cron/cron.php
View File

@@ -9,10 +9,15 @@ if (php_sapi_name() !== 'cli') {
}
// Only one run at a time. Autopay charges cards, so an overlapping run (the previous
// run still going when the next one fires) could bill the same invoice twice. The
// handle is held for the life of the process and released when it exits.
$cron_lock_handle = fopen(sys_get_temp_dir() . '/itflow_cron.lock', 'c');
if ($cron_lock_handle === false || !flock($cron_lock_handle, LOCK_EX | LOCK_NB)) {
// run still going when the next one fires) could bill the same invoice twice. The lock
// file is named per install so separate ITFlow instances on one host don't block each
// other, and the handle is held for the life of the process, released when it exits.
$cron_lock_file = sys_get_temp_dir() . '/itflow_cron_' . md5(__DIR__) . '.lock';
$cron_lock_handle = fopen($cron_lock_file, 'c');
if ($cron_lock_handle === false) {
die("Cannot open the cron lock file at $cron_lock_file - check permissions and open_basedir.\n");
}
if (!flock($cron_lock_handle, LOCK_EX | LOCK_NB)) {
die("Cron is already running - exiting.\n");
}