Error when no rollbar token for local testing

[mnwalker]

Just updated composer and got a newer rollbar version which has started throwing an error constantly.

`Unresolvable dependency resolving [Parameter #0 [ array $config ]] in class Rollbar\RollbarLogger

at vendor/laravel/framework/src/Illuminate/Container/Container.php:1141
1137▕ protected function unresolvablePrimitive(ReflectionParameter $parameter)
1138▕ {
1139▕ $message = "Unresolvable dependency resolving [$parameter] in class {$parameter->getDeclaringClass()->getName()}";
1140▕
➜ 1141▕ throw new BindingResolutionException($message);
`

Have found the solution is to simply include the ROLLBAR_TOKEN value in my .env file.

But this breaks a common usecase that was previously recommended, not sure if actually by rollbar or elsewhere - which was not setting the token to avoid logging errors in development environment.

Looking into docs it seems that conditional loading is a workaround, but this is adding lots of extra complexity to something that previously just worked.

[linear]

SDK-448 Error when no rollbar token for local testing

[brianr]

Thanks @mnwalker , we'll take a look at this.

[ampersand]

This appears to be caused by the boot method in the RollbarServiceProvider. It now has a parameter for the RollbarLogger, but when the token is not set the register method doesn't run.

I'm not certain if this is the best fix, but here is what I came up with.

    public function boot(): void
    {
        if ($this->stop() === true) {
            return;
        }

        $this->app->make(RollbarLogger::class);

        // Set up telemetry if it is enabled.
        if (null !== Rollbar::getTelemeter()) {
            $this->setupTelemetry($this->getConfigs($this->app));
        }
    }

[JerryVerhoef]

@ampersand I came to the same solution.... I made a PR for this...

[danielmorell]

We released a fix for this from @JerryVerhoef in v8.1.1 just now. Thanks for your patience and assistance!