vaadin / observability-kit

Other
5 stars 2 forks source link

fix: remove dynamic parts from root span name #118

Closed sissbruecker closed 1 year ago

sissbruecker commented 1 year ago

Description

The logic in StreamResourceHandlerInstrumentation for removing the parametrized URL parts does not work properly ATM:

Bildschirmfoto 2022-09-15 um 14 41 05

This changes the logic to generate a new path based on the filename instead of trying to fix the existing one. That might not be 100% in all cases, but it's consistent, and users probably don't care that much about the actual URL. This also sets the http.route to the generalized URL, and http.target to the actual path.

Bildschirmfoto 2022-09-15 um 14 26 12