Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Config] Add Use Path as Transaction Name and Add Transaction Name Groups #305

Open
jackshirazi opened this issue Jun 28, 2024 · 0 comments

Comments

@jackshirazi
Copy link
Contributor

Classic agent link

What it does

Use Path as Transaction Name makes transaction names of unsupported or partially-supported frameworks be in the form of $method $path instead of just $method.
Add Transaction Name Groups groups transaction names that contain dynamic parts, using a wildcard expression. For example, the pattern GET /user/*/cart would consolidate transactions, such as GET /users/42/cart and GET /users/73/cart into a single transaction name GET /users/*/cart

Implementation difficulty, location, and dynamism

  • I think this is easiest done in the exporter - look for http spans with just "$method" as name, and add a (partial) path if available; and do wildcard matching and change the name for the groups. The support for queue transaction names can be done too, but if any difficulty (eg identifying queue spans), then drop or shift to phase 2. Alternatively, we could make this generic and apply to ALL spans
  • It can be contributed upstream
  • It can be dynamic
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant