Правильное выполнение оболочки в PHP


Проблема

Я использовал функцию, которая использовала proc_open() для вызова команд оболочки. Похоже, то, как я делал STDIO, было неправильным и иногда приводило к блокировке PHP или целевой команды. Это исходный код:

function execute($cmd, $stdin=null){
    $proc=proc_open($cmd,array(0=>array('pipe','r'),1=>array('pipe','w'),2=>array('pipe','w')),$pipes);
    fwrite($pipes[0],$stdin);                fclose($pipes[0]);
    $stdout=stream_get_contents($pipes[1]);  fclose($pipes[1]);
    $stderr=stream_get_contents($pipes[2]);  fclose($pipes[2]);
    return array( 'stdout'=>$stdout, 'stderr'=>$stderr, 'return'=>proc_close($proc) );
}

Это работает большую часть времени , но этого недостаточно, я хочу, чтобы это работало всегда.

Проблема заключается в stream_get_contents() блокировке, если буферы STDIO превышают 4k данных.

Тестовый случай

function out($data){
    file_put_contents('php://stdout',$data);
}
function err($data){
    file_put_contents('php://stderr',$data);
}
if(isset($argc)){
    // RUN CLI TESTCASE
    out(str_repeat('o',1030);
    err(str_repeat('e',1030);
    out(str_repeat('O',1030);
    err(str_repeat('E',1030);
    die(128); // to test return error code
}else{
    // RUN EXECUTION TEST CASE
    $res=execute('php -f '.escapeshellarg(__FILE__));
}

Мы выводим строка дважды в STDERR и STDOUT с общей длиной 4120 байт (более 4k). Это приводит к блокировке PHP с обеих сторон.

Решение

По-видимому, stream_select() - это правильный путь. У меня есть следующий код:

function execute($cmd,$stdin=null,$timeout=20000){
    $proc=proc_open($cmd,array(0=>array('pipe','r'),1=>array('pipe','w'),2=>array('pipe','w')),$pipes);
    $write  = array($pipes[0]);
    $read   = array($pipes[1], $pipes[2]);
    $except = null;
    $stdout = '';
    $stderr = '';
    while($r = stream_select($read, $write, $except, null, $timeout)){
        foreach($read as $stream){

            // handle STDOUT
            if($stream===$pipes[1])
/*...*/         $stdout.=stream_get_contents($stream);

            // handle STDERR
            if($stream===$pipes[2])
/*...*/         $stderr.=stream_get_contents($stream);
        }

        // Handle STDIN (???)
        if(isset($write[0])) ;

// the following code is temporary
$n=isset($n) ? $n+1 : 0; if($n>10)break; // break while loop after 10 iterations

    }
}

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

Тем не менее, описанный выше подход все еще зависает на stream_get_contents(). Я совершенно не уверен, что делать/пытаться дальше.

Кредиты

Решение было предложено Якобом Трулсеном, а также обнаружением первоначальной проблемы. Подсказка 4k также была его идеей. До этого я был озадачен тем, почему функция работала нормально (не знал, что все зависит от размера буфера).

Author: Christian, 2011-05-16

4 answers

Ну, кажется, прошел год, и я забыл, что это все еще не сделано!

Тем не менее, я закончил этот беспорядок в хорошем классе PHP, который вы можете найти на Github.

Основная оставшаяся проблема заключается в том, что чтение STDERR приводит к блокировке PHP-скрипта, поэтому он был отключен.

С другой стороны, благодаря событиям и некоторому приятному кодированию (я надеюсь!), можно фактически взаимодействовать с выполняемым процессом (отсюда и название класса, InterExec). Таким образом, вы можете иметь стиль бота поведение в PHP.

 5
Author: Christian, 2012-12-17 11:46:33

Вы пропустили это примечание в руководстве PHP для stream_select():

Когда возвращается функция stream_select(), массивы чтения, записи и исключения изменяются, чтобы указать, какие ресурсы потока фактически изменили статус.

Вам нужно заново создавать массивы перед каждым вызовом stream_select().

В зависимости от процесса, который вы открываете, это может быть причиной того, что ваш пример все еще блокируется.

 3
Author: David Anderson, 2013-11-06 23:05:15
while($r = stream_select($read, $write, $except, null, $timeout)){

Насколько я знаю, это установит $r на количество измененных потоков, которое может быть равно 0, и цикл больше не будет продолжаться. Я бы лично перекодировал это, как описано в руководстве по PHP:

while(false !== ($r = stream_select($read, $write, $except, null, $timeout))){

Что касается вашего стандартного ввода-кода, если ваш процесс не является интерактивным, то стандартный ввод может не потребоваться. Какой процесс вы выполняете?

 0
Author: Jay, 2011-05-18 06:43:40

Вся проблема с зависанием в stream_get_contents заключается в том, как создается процесс. Правильный способ - открыть стандартный вывод в режиме чтения/записи канала, например:

$descriptor = array (0 => array ("pipe", "r"), 1 => array ("pipe", "rw"), 2 => array ("pipe", "rw"));
//Open the resource to execute $command
$t->pref = proc_open($command,$descriptor,$t->pipes);
//Set STDOUT and STDERR to non-blocking 
stream_set_blocking ($t->pipes[0], 0);
stream_set_blocking ($t->pipes[1], 0);

Очевидно, что когда stream_get_contents хочет прочитать канал STDOUT, ему нужен режим чтения. Та же ошибка с зависанием/замораживанием/блокировкой находится в этом хорошем классе https://gist.github.com/Arbow/982320

Затем блокировка исчезает. Но читать ничего не значит читать.

 0
Author: OSP, 2013-04-20 13:11:56