chore(medusa): Add handler path to the http tracing to be able to group by (#10835)

RESOLVES FRMW-2835

In this PR, we trace HTTP requests using the route pattern and not the request URL. This allows us to aggregate all HTTP requests under a single route.

In terms of implementation, we have to self find the route for a given request, since there is no API in express to do the same and we begin tracing even before the request is handed over to Express.

Since, the route matching happens via RegExp matches, we ensure that this does not add much performance overhead. The matching takes between 0.8ms-1.5ms for various different routes

Co-authored-by: Harminder Virk <1706381+thetutlage@users.noreply.github.com>

This commit is contained in:

Adrien de Peretti

2025-01-07 13:16:20 +00:00

committed by

GitHub

co-authored by

Harminder Virk

parent 5216ad2f15

commit 899b1fba4a

3 changed files with 61 additions and 4 deletions

									
										.changeset/itchy-walls-call.md
									
		+5
		
												View File
												
				@@ -0,0 +1,5 @@

				---

				"@medusajs/medusa": patch

				---

				chore(medusa): Add handler path to the http tracing to be able to group by