sentry-symfony icon indicating copy to clipboard operation
sentry-symfony copied to clipboard

Improve parameterization of transaction names

Open smeubank opened this issue 2 years ago • 2 comments

Environment

How do you use Sentry? Sentry SaaS (sentry.io)

Which SDK and version? sentry-symfony

Steps to Reproduce

similar to https://github.com/getsentry/sentry-laravel/issues/562

Expected Result

Parameterized transactions as often as possible

Actual Result

seems symfony only sends raw URLs for everything

smeubank avatar Aug 01 '22 09:08 smeubank

we should open a convo with Symfony that they add something to help us access their routing properly

smeubank avatar Aug 04 '22 12:08 smeubank

This issue has gone three weeks without activity. In another week, I will close it.

But! If you comment or otherwise update it, I will reset the clock, and if you label it Status: Backlog or Status: In Progress, I will leave it alone ... forever!


"A weed is but an unloved flower." ― Ella Wheeler Wilcox 🥀

github-actions[bot] avatar Aug 26 '22 00:08 github-actions[bot]

This issue has gone three weeks without activity. In another week, I will close it.

But! If you comment or otherwise update it, I will reset the clock, and if you label it Status: Backlog or Status: In Progress, I will leave it alone ... forever!


"A weed is but an unloved flower." ― Ella Wheeler Wilcox 🥀

github-actions[bot] avatar Nov 02 '22 00:11 github-actions[bot]

So after speaking with various folks during SymfonyCon, we'll likely use the route name as the default source for transaction names. While the _controller property could also be an option, there is the use case of https://symfony.com/doc/4.1/templating/render_without_controller.html which ends up being not very useful.

See #547 for a former discussion

cleptric avatar Nov 21 '22 15:11 cleptric