FriendsOfFlarum Sentry

yoyo

Hello, i want to change debug level to errors only, how do I do it and what file should I change?
right now I am getting too many info/warning entries into Sentry. Thank you

datitisev

yoyo Those info & warnings can sometimes break things, which is why I chose to report them. You should be able to ignore them in Sentry, I think.

datitisev

0.5.0

Support Flarum beta 13
- Requires beta 12 or later
Update JS Sentry package

Updating

composer require fof/sentry
php flarum cache:clear

danielunited

Useful extension! Should the Sentry project be set to PHP or Laravel?

datitisev

danielunited It does not matter. That is only a visual icon in the Sentry dashboard. This extension reports PHP & JS (if enabled).

datitisev

0.6.0

Support Flarum beta 14
Update JS & PHP Sentry packages

Updating

composer require fof/sentry
php flarum cache:clear

IanM

0.7.0

Flarum beta 15 ready

davwheat

0.8.0

Adds support for Beta 16 (and 17, for now)
Add backend JS performance tracking (courtesy of @luceos)
Add frontend JS performance tracking
Bump Sentry to latest version

Updating

composer update fof/sentry:"*"
php flarum cache:clear

MikeJones

davwheat extiverse helper tells em this is not compatible with beta 16 yet.... Is this just a bug in extiverse helper?

davwheat

MikeJones

It's because there were many breaking changes between B15 and B16 so any extension that does not explicitly require flarum/core with >= 0.1.0.beta-16 does not show as compatible, even if it is.

MikeJones

davwheat ok so it should work if I update?

davwheat

MikeJones

Yes.

luceos

MikeJones there are a couple of issues with the latest version that introduces performance monitoring. I have planned to take a look at it.

Performance is dramatically impacted on larger communities, which might relate to:
Hitting a 404 page causes query timeouts/locks just try hitting <yourUrl>/somethingrandom

Would be great if you could confirm the same happens for you 👍️

MikeJones

luceos what should I look for when upgrading, just slower performance when navigating page to page?

Any specific tests you need me to run once I upgrade?

IanM

1.0.0

Flarum 1.0 compatibility (cc MikeJones )
Resolve poor performance issues luceos

Installation

composer require fof/sentry:"*"
php flarum cache:clear

MikeJones

Just checking if there are plans to update this extension? Its my last one before upgrade to stable. If not no worries, I may just uninstall it.

IanM

MikeJones we're close.. It's updated, but I'm currently trying to figure out one remaining issue before releasing it. Darn 🐛

MikeJones

IanM awesome no rush! I got a wedding coming up so maybe I will wait until after to update ;-)

MikeJones

IanM dang, you guys are all truly amazing!

pkernstock

That's cool - thanks @IanM!

I've installed Sentry after the update to Flarum 1.0 on the weekend, but once I enabled backend performance monitoring sending emails seems to be affected on my setup? Not sure if it's related to any specific extension in use.

The stacktrace (the first 1/3, as it's quite long) was:

[2021-07-01 17:55:05] flarum.ERROR: Exception: Error: Call to protected method FoF\Sentry\SentryServiceProvider::handleError() from scope Sentry\ErrorHandler in /vwr/www/flarum/vendor/sentry/sentry/src/ErrorHandler.php:303
Stack trace:
#0 /vwr/www/flarum/vendor/fof/pretty-mail/src/BladeCompiler.php(28) : eval()'d code(40): Sentry\ErrorHandler->handleError()
#1 /vwr/www/flarum/vendor/fof/pretty-mail/src/BladeCompiler.php(28): eval()
#2 /vwr/www/flarum/vendor/fof/pretty-mail/src/Mailer.php(74): FoF\PrettyMail\BladeCompiler::render()
#3 /vwr/www/flarum/vendor/flarum/core/src/Api/Controller/SendTestMailController.php(43): FoF\PrettyMail\Mailer->raw()
#4 /vwr/www/flarum/vendor/flarum/core/src/Http/RouteHandlerFactory.php(41): Flarum\Api\Controller\SendTestMailController->handle()
#5 /vwr/www/flarum/vendor/flarum/core/src/Http/Middleware/ExecuteRoute.php(27): Flarum\Http\RouteHandlerFactory->Flarum\Http\{closure}()
#6 /vwr/www/flarum/vendor/laminas/laminas-stratigility/src/Next.php(51): Flarum\Http\Middleware\ExecuteRoute->process()
#7 /vwr/www/flarum/vendor/reflar/recache/src/Middlewares/OptimizeImages.php(43): Laminas\Stratigility\Next->handle()
#8 /vwr/www/flarum/vendor/laminas/laminas-stratigility/src/Next.php(51): Reflar\ReCache\Middlewares\OptimizeImages->process()
#9 /vwr/www/flarum/vendor/fof/sentry/src/Middleware/HandleErrorsWithSentry.php(39): Laminas\Stratigility\Next->handle()
#10 /vwr/www/flarum/vendor/laminas/laminas-stratigility/src/Next.php(51): FoF\Sentry\Middleware\HandleErrorsWithSentry->process()
#11 /vwr/www/flarum/vendor/kilowhat/flarum-ext-audit-free/src/Middlewares/SetLoggerActor.php(28): Laminas\Stratigility\Next->handle()
#12 /vwr/www/flarum/vendor/laminas/laminas-stratigility/src/Next.php(51): Kilowhat\Audit\Middlewares\SetLoggerActor->process()
#13 /vwr/www/flarum/vendor/flarum/core/src/Api/Middleware/ThrottleApi.php(33): Laminas\Stratigility\Next->handle()
#14 /vwr/www/flarum/vendor/laminas/laminas-stratigility/src/Next.php(51): Flarum\Api\Middleware\ThrottleApi->process()

IanM

pkernstock thanks for letting me know, should be pretty straightforward to fix. I’ll get to it as soon as I can 👍🏻

luceos

IanM I committed the change, feel free to test/tag 😉

pkernstock feel free to give dev-master a spin so that @IanM only needs to tag 😬

IanM

1.0.1

Fix method visibility (thanks for the fast PR @luceos ) pkernstock

Installation

composer require fof/sentry:"*"

« Previous Page Next Page »