Как запретить одновременный запуск PHP скрипта?

Question

[aopil]

Есть файл test.php
В этом файле я добавил вот такой код:

public function __construct()
{
       $lock = fopen('lock_file.lock', 'w');
        if (!($lock && flock($lock, LOCK_EX | LOCK_NB))) {
            exit('already running');
        }
}

Теперь делаем простую вещь: переходим по адресу 111.222.333/test.php тем самым запускаем скрипт, далее он начинает выполняться, тем временем мы открываем новую вкладку и делаем еще один запуск - он УСПЕШНО выполняется без ошибок.
обе вкладки грузятся секунд 30-40 и по завершению загрузки не выдают ошибок, т.е. в обоих случаях скрипт выполнился.

ВОПРОС: Как в итоге запретить повторный запуск скрипта??? Т.е. если скрипт уже выполняет задачу, чтобы даже если мы пытаемся запустить его еще раз он сразу завершал попытку и не грузился 30 секунд.

Comments

[Одиночка Айс]

Зачем блокировать? Есть файл - выходим из скрипта, нет - запускаем и создаем файл

[aopil]

Одиночка Айс, не знаю... хоть видео записывай, но у меня не работает такой метод.

Точнее так: запуск 2 сессий дает следующий результат:
В обоих случаях страница грузится секунд 30, затем выдается результат:
1 сессия успешно, а вторая с ошибкой (то что требуется)

у меня вопрос, на какой хрен он грузит страницу 30 секунд если сразу должен завершить работу....................... у меня горит уже честно с этого :(

[Дмитрий Свиридов]

aopil, у тебя в скриптах SESSION используется? session_start.

[Дмитрий]

Да проверка косячит
Запускает ли cron задачу если она уже выполняется?

[aopil]

Дмитрий Свиридов,

<?php
set_time_limit(0);
ini_set('error_reporting', E_ALL);
ini_set('display_errors', 1);
ini_set('display_startup_errors', 1);

include("config.php");

class script {
/**/
}
$object = new Script($DSN, $DB, $DB_PASS, $OPTIONS);

примерно так выглядит код.

[Сергей Соколов]

Сколько процессоров в сервере?
Для обработки PHP используется php-fpm?
Сколько рабочих процессов php-fpm запущено и сколько максимум их разрешено в настройках php-fpm?
Есть ли база данных, MySQL. Может, установлен Redis, memcached, APC?

[Ninazu]

aopil Ждешь 30 секунд, потому что это время жизни скрипта/сессии
Добавь в самое начало скрипта
session_write_close();
Или отключи
session.autostart=off

Answer 1

[Дмитрий Свиридов]

Если у тебя в PHP скриптах используются сессии, то с одного браузера ты параллельно скрипты не запустишь, даже из разных вкладок браузера. Скрипты выполняются последовательно. Попробуй через инкогнито или один скрипт дернуть одним браузером, другой - другим.

Comments

[Сергей Соколов]

не думаю, что это как-то связано с сессиями. Скорее влияет количество рабочих процессов php-fpm и прочие настройки: выполняются ли все запросы php-fpm (псевдо-)параллельно, или строго в один поток. Во втором случае всем придётся ждать пока отыграет запущенная ранее долгоиграющая задача.

[Nikita Krasnikov]

Автор это твоя причина. Чел дело говорит.

Answer 2

[Алексей Уколов]

Лучше всего, вместо костылей из файлов, использовать готовый инструмент: https://www.php.net/manual/ru/class.syncmutex.php

Comments

[aopil]

Алексей Уколов Видимо отпадает:
Fatal error: Class 'SyncMutex' not found in

[Вячеслав Барсуков]

PECL sync >= 1.0.0
стоит?

[aopil]

Вячеслав Барсуков, ну если он с php не ставится сам, то видимо нет) Вопрос поможет ли оно в моем случае... я перечитал много тем подобных, где все пытаются решать проблему созданием файла в итоге у меня почему-то не получается :(

[Алексей Уколов]

Поможет, конечно, он для этого и создан.

Answer 3

[Rsa97]

public function __construct()
{
  if (file_exists('lock_file.lock')) {
    exit('already running');
  }
  touch('lock_file.lock');
}

public function __destruct()
{
  unlink('lock_file.lock');
}

Comments

[aopil]

Rsa97 запуск 2 сессий дает следующий результат:
В обоих случаях страница грузится секунд 30, затем выдается результат:
1 сессия успешно, а вторая с ошибкой (то что требуется)

у меня вопрос, на какой хрен он грузит страницу 30 секунд если сразу должен завершить работу....................... у меня горит уже честно с этого :(

вот так пробую:

if (!file_exists(__DIR__ . '/lock_file.lock')) {
            touch(__DIR__ . '/lock_file.lock');
        } else {
            exit('test');
        }

[Сергей]

aopil, естественно. Сессия у вас на файлах. И пока первый запуск не отработает, второй даже не начнётся.
Это будет работать только под разными браузерами, пользователями

[Сергей Соколов]

aopil, возможно, у вас php выполняется в один поток (однопроцессорный сервер и таковы настройки php-fpm) – поэтому второй в любом случае вынужден ждать пока отработает первый.

Answer 4

[Сергей Соколов]

Попробуйте как-то так:

const FLAGFILE = 'lock_file.lock';
private $myrun = true;

public function __construct()
{
    if (file_exists(self::FLAGFILE) {
        $this->myrun = false;
        exit('Lock file exists');
    }
    touch(self::FLAGFILE);
}

function __destruct ()
{
    if ($this->myrun) unlink(self::FLAGFILE);
}

Comments

[aopil]

Сергей Соколов , сначала тестировал - сразу (за 1 секунду выдавал - Lock file exists) то что и требуется!

Потыкался пару раз - теперь опять грузит страничку секунд 20-30 и потом только выдает что мол - Lock file exists, хотя чего он ждет 20-30 секунд, если я запустил только что 1 сессию и файл создался, потом запускаю сразу еще одному и она грузится.... спустя кучу времени завершает выполнение с этим текстом.

Действие срабатывает - да, это хорошо, но почему оно ждет время чтобы сработать? :(

[Дмитрий Свиридов]

aopil, в ответе на вопрос тебе ответил (попытался ответить)

[aopil]

Сергей Соколов вроде работает, можете объяснить смысл private $myrun = true; в этом коде?

[Сергей Соколов]

aopil, $myrun (да/нет) – «мой запуск» – в этот ли раз можно будет выполнить долгоиграющий код, или он уже выполняется в другом экземпляре, а «мне» надо по-быстрому свернуться.

Это нужно, чтобы в __destruct() понять, нужно ли удалять файл флаг.

Но надо всё проверить на практике, т.к. есть сомнения: если exit() сработает уже в конструкторе — будет ли таки создан объект и вызовется ли тут же его дестрактор.

[Руслан Хорошкевич]

<?php
set_time_limit(0);
ini_set('error_reporting', E_ALL);
ini_set('display_errors', 1);
ini_set('display_startup_errors', 1);

include("config.php");

$myPID = getmypid();
if(is_readable('lock_file.lock')) {
  $pid = file_get_contents('lock_file.lock');
  // проверяем наличие запущенного процесса
  exec('ps --pid  "'. $pid .'"|grep "php"|grep "'. $pid .'"')
  if(!$return_var) {
    // скрипт уже работает
    exit('Lock file exists');
}

file_put_contents('lock_file.lock', $myPID);

class script {
/**/
}
$object = new Script($DSN, $DB, $DB_PASS, $OPTIONS);

unlink('lock_file.lock');

Answer 5

[Олег]

с вашим подходом можно решить только задачу, чтобы они работали последовательно.
если задача, чтобы второй скрипт досрочно завершил работу, то нужен
семафор (скрипт работает).
Можно проверять существование определенного файла и если он есть, то выход.
Но не забываем, что из-за ошибки файл семафора может не удалиться по завершению скрипта.
Как вариант имя зависит от даты и текущего часа. В этом случае при сбое, на следующий час исполнение автоматически продолжиться.

Comments

[aopil]

Олег мне очень интересно, а что я использую? Я же показал пример.. моего кода и расписал как он работает (точнее не работает) и рассказал то, что я хочу, а именно - если файл существует (значит скрипт работает) то если я запускаю его снова чтобы он сразу завершал работу, а сейчас он просто продолжает, можно хоть 30 сессий запустить и код из моего вопроса не помогает, не знаю почему :(

[Олег]

aopil , в ответе RSA пример, того что я Вам написал.
А у Вас не проверка существования, а захват в монопольное пользование существующего файла.
Несколько фактов. В таймлимит для пхп не входит время ожидания захвата ресурсов.
flock() ждет до тех пор, пока файл не освободится. Он не приводит к ошибке, типа ресурс занят другим.

Answer 6

[Andrey Uteshev]

check pid.