stayallive / wp-sentry

A (unofficial) WordPress plugin reporting PHP and JavaScript errors to Sentry.
https://wordpress.org/plugins/wp-sentry-integration/
MIT License
305 stars 48 forks source link

Bug: PHP errors are not getting reported anymore #91

Closed kkmuffme closed 3 years ago

kkmuffme commented 3 years ago

Sentry is not triggering/recording errors since 4.4.1 (maybe 4.4.0) anymore.

In 4.3.0 I got sentry errors for:

call_this();
exit;

or

echo $helloworld;
exit;

In 4.4.1 they wont show up in sentry.

stayallive commented 3 years ago

Can you confirm that downgrading fixes the problem?

kkmuffme commented 3 years ago

Yes, it works in 4.3.0, I just tested it again.

stayallive commented 3 years ago

Hmm, I have been trying to reproduce this but when using your code in header.php of the Twenty Seventeen theme I get the reports sent to Sentry every time.

Can you tell me a bit more about your PHP version and environment (Linux/Windows) and where exactly you are placing that code?

The code placement should not matter as long as you don't move it between testing between 4.3.0 and 4.4.x of course but just wanting to make sure.

kkmuffme commented 3 years ago

Just updated to 4.5.0 and it works again.

Looks like a bug in the sentry lib used in 4.4.0/4.4.1.

stayallive commented 3 years ago

Okay interesting, but glad it's working at least 👌