BEAR\Package\Provide\Error\LogRef

Complexity

Total Complexity

5

Size/Duplication

Total Lines	23
Duplicated Lines	0 %

Importance

Changes	1
Bugs	0	Features	0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	log()	0	8	3
A	__toString()	0	3	1
A	__construct()	0	4	1

<?php

declare(strict_types=1);

namespace BEAR\Package\Provide\Error;

use BEAR\AppMeta\AbstractAppMeta;
use BEAR\Sunday\Extension\Router\RouterMatch;
use Stringable;
use Throwable;

use function file_put_contents;
use function is_link;
use function is_writable;
use function sprintf;
use function symlink;
use function unlink;

final class LogRef implements Stringable
{
    private string $ref;

    public function __construct(Throwable $e)
    {
        // phpcs:ignore SlevomatCodingStandard.Namespaces.ReferenceUsedNamesOnly.ReferenceViaFallbackGlobalName
        $this->ref = hash('crc32b', $e::class . $e->getMessage() . $e->getFile() . $e->getLine());
    }

    public function __toString(): string
    {
        return $this->ref;
    }

    public function log(Throwable $e, RouterMatch $request, AbstractAppMeta $appMeta): void
    {
        $logRefFile = sprintf('%s/logref.%s.log', $appMeta->logDir, $this->ref);
        $log = (string) new ExceptionAsString($e, $request);
        @file_put_contents($logRefFile, $log);

It seems like you do not handle an error condition for file_put_contents(). This can introduce security issues, and is generally not recommended.

        $linkFile = sprintf('%s/last.logref.log', $appMeta->logDir);
        is_link($linkFile) && is_writable($linkFile) && @unlink($linkFile);
        @symlink($logRefFile, $linkFile);

It seems like you do not handle an error condition for symlink(). This can introduce security issues, and is generally not recommended.

    }
}