Rethrow Loses Stack Trace (ARCH7154)

Key php.logging.rethrow-without-previous · Severity warning · Category logging

Throwing a new exception inside a handler without passing the original as the previous argument destroys the root cause and stack trace.

Incorrect

<?php
try {
    $this->client->request($url);
} catch (TransportException $e) {
    throw new ServiceException('Upstream call failed');
}

Correct

<?php
try {
    $this->client->request($url);
} catch (TransportException $e) {
    throw new ServiceException('Upstream call failed', 0, $e);
}

Fix

Pass the original exception as the previous (third) argument when throwing the new one.