Reporting a RequestException in a try catch block does not log an error

[bsterckx]

Laravel Version

12.39.0

PHP Version

8.3.26

Database Driver & Version

Mariadb 10.11

Description

When a caught RequestException gets reported in a try catch block via the report($e) function no error gets logged. The issue is not there in 12.37.0 but is there in v12.38.1 & v12.39.0. This was caught because of a failing test and seems like a small backwards incompatible change.

• in v12.38.1 because of PR57759
• in v12.39.0 because of PR57767

Steps To Reproduce

it('can report a caught request exception', function () {
    Http::fake([
        'https://test.example.com/fake-endpoint' => Http::response('test', 500),
    ]);

    Log::shouldReceive('error')->once();

    try{
        Http::get('https://test.example.com/fake-endpoint')->throw();
    } catch(RequestException $e){
        // do something
        report($e);
    }
});

[cosmastech]

Ooof. I think the report() method needs to return false here.

[bsterckx]

Unsure what you mean by that? The return type of the report function is void so how would it return false?
The issue only exists for RequestException other types of exceptions can be reported in a catch block without any problem.

[cosmastech]

Unsure what you mean by that? The return type of the report function is void so how would it return false? The issue only exists for RequestException other types of exceptions can be reported in a catch block without any problem.

Sorry, I meant that I caused the problem. This should be resolved in the next release. #57847