owen-it / laravel-auditing

src/Resolvers/IpAddressResolver.php

<?php

namespace OwenIt\Auditing\Resolvers;

use Illuminate\Support\Facades\Request;
use OwenIt\Auditing\Contracts\Auditable;
use OwenIt\Auditing\Contracts\Resolver;

class IpAddressResolver implements Resolver
{
    public static function resolve(Auditable $auditable): string
    {
        return $auditable->preloadedResolverData['ip_address'] ?? Request::ip();

The expression return $auditable->preloadedResolverData['ip_address'] ?? Illuminate\Support\Facades\Request::ip() could return the type null which is incompatible with the type-hinted return string. Consider adding an additional type-check to rule them out.

Accessing preloadedResolverData on the interface OwenIt\Auditing\Contracts\Auditable suggest that you code against a concrete implementation. How about adding an instanceof check?

    }
}