使用HTTP 客户端内存不释放

[peimengc]

Laravel Version

11.44.2

PHP Version

8.2

Database Driver & Version

No response

Description

在循环里面使用Http客户端发送请求,内存一直在增长
使用GuzzleHttp并不会

Steps To Reproduce

 $uri = 'https://jsonplaceholder.typicode.com/posts';
    do {
        $this->info('开始内存: ' . memory_get_usage());

       //(new \GuzzleHttp\Client())->get($uri);  //内存并不会增长
        \Illuminate\Support\Facades\Http::get($uri); //内存一直在增长
        sleep(1);
    } while (true);

[AndrewMast]

为了将来参考，请用英语发布您的问题
_{For future reference, please post your issues in English.}

Memory is not released when using HTTP client

Laravel Version

11.44.2

PHP Version

8.2

Database Driver & Version

No response

Description

When using the Http client to send requests in a loop, the memory keeps growing.
Using GuzzleHttp does not do this.

Steps To Reproduce

$uri = 'https://jsonplaceholder.typicode.com/posts';

do {
    $this->info('Memory: ' . memory_get_usage());

    //(new \GuzzleHttp\Client())->get($uri);  // Memory does not grow
    \Illuminate\Support\Facades\Http::get($uri); // Memory does grow

    sleep(1);
} while (true);

[timacdonald]

I created a fresh Laravel application:

composer create-project laravel/laravel

I then added the following artisan command to test the issue:

<?php

// file: routes/console.php

use Illuminate\Support\Facades\Artisan;

Artisan::command('dev', function () {
    $uri = 'https://jsonplaceholder.typicode.com/posts';

    do {
        \Illuminate\Support\Facades\Http::get($uri);

        gc_collect_cycles();
        echo 'Memory: '.memory_get_usage().PHP_EOL;
        sleep(1);
    } while (true);
});

The result is consistent memory usage:

A few things with your initial script that could have caused issues:

• I would recommend using echo instead of any other means to output memory consumption. This reduces the surface area of things that could be using memory.

- $this->info('Memory: ' . memory_get_usage());
+ echo 'Memory: ' . memory_get_usage();

• You always want to trigger garbage collection before checking memory usage. PHP only releases memory when it needs to. So by design, PHP will often seem like it is consuming memory that may be available for garbage collection.

+ gc_collect_cycles();
echo 'Memory: ' . memory_get_usage();

[rodrigopedra]

You were checking for allocated memory, not consumed memory:

For reference, it is actually the other way around.

• memory_get_usage(true): total memory allocated from system
• memory_get_usage() or memory_get_usage(false): only the used memory is reported

reference: https://www.php.net/manual/en/function.memory-get-usage.php

But the bottom line is still the same as @timacdonald described.

[timacdonald]

Whoops! Updated the comment and the results are as I would expect:

[noefleury]

Interesting ! But why does the Guzzle way is not acting the same in the base example ? It don't look like there is some garbage collector usage difference but the memory is not increasing.