midcom_exception_handler::process_actions() - Code Metrics - Inspection of "Refactor for greater flexibility, less superglobal..." - flack/openpsa - Measure and Improve Code Quality continuously with Scrutinizer

Passed
Push — master ( 437a9c...91e741 )

by Andreas
created 2024-06-25 15:31 UTC
midcom_exception_handler::process_actions() A

↳ Parent: midcom_exception_handler
Complexity

Conditions	5
Paths	7
Size

Total Lines	22
Code Lines	11
Duplication

Lines	0
Ratio	0 %
Code Coverage

Tests	0
CRAP Score	30
Importance

Changes
Metric	Value
cc	5
eloc	11
nc	7
nop	3
dl	0
loc	22
ccs	0
cts	12
cp	0
crap	30
rs	9.6111
c	0
b	0
f	0
<?php
/**
 * @package midcom
 * @author The Midgard Project, http://www.midgard-project.org
 * @copyright The Midgard Project, http://www.midgard-project.org
 * @license http://www.gnu.org/licenses/lgpl.html GNU Lesser General Public License
 */

use Symfony\Component\HttpFoundation\Response;
use Monolog\Logger;
use Monolog\Handler\StreamHandler;
use Symfony\Component\EventDispatcher\EventSubscriberInterface;
use Symfony\Component\HttpKernel\KernelEvents;
use Symfony\Component\HttpKernel\Event\ExceptionEvent;
use Symfony\Component\HttpFoundation\ServerBag;

/**
 * Class for intercepting PHP errors and unhandled exceptions. Each fault is caught
 * and converted into Exception handled by midcom_exception_handler::show() with
 * code 500 thus can be customized and make user friendly.
 *
 * @package midcom
 */
class midcom_exception_handler implements EventSubscriberInterface
{
    private Throwable $error;

    public static function getSubscribedEvents()
    {
        return [
            KernelEvents::EXCEPTION => ['handle']
        ];
    }

    /**
     * Render an error response.
     *
     * This will display a simple HTML Page reporting the error described by $httpcode and $message.
     * The $httpcode is also used to send an appropriate HTTP Response.
     *
     * The error pages can be customized by creating style elements named midcom_error_$httpcode.
     *
     * For a list of the allowed HTTP codes see the Response::HTTP_... constants
     */
    public function handle(ExceptionEvent $event)
    {
        $this->error = $event->getThrowable();

        $httpcode = $this->error->getCode();
        $message = $this->error->getMessage();
        debug_print_r('Exception occurred: ' . $httpcode . ', Message: ' . $message . ', exception trace:', $this->error->getTraceAsString());

        if (!array_key_exists($httpcode, Response::$statusTexts)) {
            debug_add("Unknown Errorcode {$httpcode} encountered, assuming 500");
            $httpcode = Response::HTTP_INTERNAL_SERVER_ERROR;
        }

        // Send error to special log or recipient as per configuration.
        $this->process_actions($event->getRequest()->server, $httpcode, $message);

        if (PHP_SAPI === 'cli') {
            throw $this->error;
        }

        $event->allowCustomResponseCode();
        $event->setResponse($this->process($httpcode, $message));
    }

    private function process(int $httpcode, string $message) : Response
    {

        if ($httpcode == Response::HTTP_FORBIDDEN) {
            return new midcom_response_accessdenied($message);
        }
        if ($httpcode == Response::HTTP_UNAUTHORIZED) {
            if ($this->error instanceof midcom_error_forbidden) {
                return new midcom_response_login($this->error->get_method());
                return new midcom_response_login($this->error->/** @scrutinizer ignore-call */ get_method());
            }

            return new midcom_response_login;
        }

        $style = midcom::get()->style;

        $style->data['error_title'] = Response::$statusTexts[$httpcode];
        $style->data['error_message'] = $message;
        $style->data['error_code'] = $httpcode;
        $style->data['error_exception'] = $this->error;
        $style->data['error_handler'] = $this;

        ob_start();
        if (!$style->show_midcom('midcom_error_' . $httpcode)) {
            $style->show_midcom('midcom_error');
        }
        $content = ob_get_clean();

        return new Response($content, $httpcode);
    }

    /**
     * Send error for processing.
     *
     * If the given error code has an action configured for it, that action will be
     * performed. This means that system administrators can request email notifications
     * of 500 "Internal Errors" and a special log of 404 "Not Founds".
     */
    private function process_actions(ServerBag $server, int $httpcode, string $message)
    {
        $error_actions = midcom::get()->config->get_array('error_actions');
        if (!isset($error_actions[$httpcode]['action'])) {
            // No action specified for this error code, skip
            return;
        }

        // Prepare the message
        $msg = "{$server->getString('REQUEST_METHOD')} request to {$server->getString('REQUEST_URI')}: ";
        $msg .= "{$httpcode} {$message}\n";
        if ($server->has('HTTP_REFERER')) {
            $msg .= "(Referrer: {$server->getString('HTTP_REFERER')})\n";
        }

        // Send as email handler
        if ($error_actions[$httpcode]['action'] == 'email') {
            $this->_send_email($msg, $error_actions[$httpcode], $server->getString('SERVER_NAME'));
        }
        // Append to log file handler
        elseif ($error_actions[$httpcode]['action'] == 'log') {
            $this->_log($msg, $error_actions[$httpcode]);
        }
    }

    private function _log(string $msg, array $config)
    {
        if (empty($config['filename'])) {
            // No log file specified, skip
            return;
        }

        if (   !is_writable($config['filename'])
            && !is_writable(dirname($config['filename']))) {
            debug_add("Error logging file {$config['filename']} is not writable", MIDCOM_LOG_WARN);
            return;
        }

        // Add the line to the error-specific log
        $logger = new Logger(__CLASS__);
        $logger->pushHandler(new StreamHandler($config['filename']));
        $logger = new midcom_debug($logger);
        $logger->log($msg, MIDCOM_LOG_INFO);
    }

    private function _send_email(string $msg, array $config, string $servername)
    {
        if (empty($config['email'])) {
            // No recipient specified, skip
            return;
        }

        if (!midcom::get()->componentloader->is_installed('org.openpsa.mail')) {
            debug_add("Email sending library org.openpsa.mail, used for error notifications is not installed", MIDCOM_LOG_WARN);
            return;
        }

        $mail = new org_openpsa_mail();
        $mail->to = $config['email'];
        $mail->from = "\"MidCOM error notifier\" <webmaster@{$servername}>";
        $mail->subject = "[{$servername}] " . str_replace("\n", ' ', $msg);
        $mail->body = "{$servername}:\n{$msg}";

        $stacktrace = $this->get_function_stack();

        $mail->body .= "\n" . implode("\n", $stacktrace);

        if (!$mail->send()) {
            debug_add("failed to send error notification email to {$mail->to}, reason: " . $mail->get_error_message(), MIDCOM_LOG_WARN);
        }
    }

    public function get_function_stack()
    {
        $stack = $this->error->getTrace();

        $stacktrace = [];
        foreach ($stack as $number => $frame) {
            $line = $number + 1;
            if (array_key_exists('file', $frame)) {
                $file = str_replace(MIDCOM_ROOT, '[midcom_root]', $frame['file']);
                $line .= ": {$file}:{$frame['line']}  ";
            } else {
                $line .= ': [internal]  ';
            }
            if (array_key_exists('class', $frame)) {
                $line .= $frame['class'];
                if (array_key_exists('type', $frame)) {
                    $line .= $frame['type'];
                } else {
                    $line .= '::';
                }
            }
            if (array_key_exists('function', $frame)) {
                $line .= $frame['function'];
            } else {
                $line .= 'require, include or eval';
            }
            $stacktrace[] = $line;
        }

        return $stacktrace;
    }
}

flack / openpsa

Push — master ( 437a9c...91e741 )

midcom_exception_handler::process_actions() A

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like
