Digitalist-Open-Cloud / Matomo-Plugin-Sentry

Log all Matomo errors to a Sentry server
https://digitalist.cloud/
GNU General Public License v3.0
4 stars 2 forks source link

The plugin '' has not been loaded. at /index.php #3

Closed mfb closed 5 years ago

mfb commented 5 years ago

When the Sentry plugin is enabled, I get

An error occurred The plugin '' has not been loaded.

when attempting to load /index.php

Findus23 commented 5 years ago

Hi,

Interestingly I had the same bug. But every time I tried to debug it and disabled and reenabled it, the error was suddenly gone. I think it has something to do with an error in this function (https://github.com/Findus23/plugin-SentryLogger/blob/master/SentryLogger.php#L22) or maybe just an old cache.

Lukas

Am 04.04.2019 um 22:44 schrieb mark burdett notifications@github.com:

When the Sentry plugin is enabled, I get

An error occurred The plugin '' has not been loaded.

when attempting to load /index.php

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub, or mute the thread.

mfb commented 5 years ago

For me, the error happens if the Sentry PHP DSN is configured. If I set it to empty string then everything works.

It also causes some 500 errors when websites load the tracker - I have 1000 events in sentry related to this.. so at least that part is working :p But unfortunately I'll need to disable the plugin if there are some 500 errors.

Findus23 commented 5 years ago

This makes sense as the plugin does nothing if the dsn is an empty string.

Maybe one of these throw an error https://github.com/Findus23/plugin-SentryLogger/blob/master/SentryLogger.php#L37-L43 And therefore stop Matomo from loading plugins and therefore working at all.

Lukas

Am 04.04.2019 um 23:07 schrieb mark burdett notifications@github.com:

For me, the error happens if the Sentry PHP DSN is configured. If I set it to empty string then everything works.

It also causes some 500 errors when websites load the tracker - I have 1000 events in sentry related to this.. so at least that part is working :p But unfortunately I'll need to disable the plugin if there are some 500 errors.

— You are receiving this because you commented. Reply to this email directly, view it on GitHub, or mute the thread.

mfb commented 5 years ago

Well, you can see the stacktrace in Sentry -

the problematic line is "currentPlugin" => Piwik::getCurrentPlugin()->pluginName,

Findus23 commented 5 years ago

Now that I think about it, fetching the list of all plugins during initialising of plugins doesn't seem that smart :)

I have removed that line in 0.2.3

mfb commented 5 years ago

Seems resolved.