fix(instrumentation-koa): handle koa routes being of type RegExp

[ramesius]

Which problem is this PR solving?

Koa route paths can be string | RegExp. RegExp is not a primitive type and thus results in the following logs from Otel when using the koa instrumentation:

• Invalid attribute value set for key: http.route
• Invalid attribute value set for key: koa.name

Short description of the changes

This change updates the typings package @types/koa__router that in-turn has been updated with the correct union type string | RegExp (See PR DefinitelyTyped/DefinitelyTyped#67194). The Koa instrumentation now properly handles the case when routes are of type RegExp.

I have also replaced the use of the deprecated SpanAttributes with Attributes.

[ramesius]

@david-luna Is this something that you could look into?

[david-luna]

@david-luna Is this something that you could look into?

I can give you feedback on this PR for now but nothing more since I don't have the required role/access for actioning on PRs.

So PR LGTM but I was wondering if its necessary to send KOA_TYPE and HTTP_ROUTE attributes if layerPath is not defined (not passed in the function call)

[blumamir]

@ramesius Thank you for the contribution. the fix LGTM.
Can you please add a test for this case?

[codecov]

Codecov Report

Merging #1754 (30b2133) into main (24ffb3b) will increase coverage by 0.00%.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##             main    #1754   +/-   ##
=======================================
  Coverage   91.42%   91.42%           
=======================================
  Files         143      143           
  Lines        7299     7301    +2     
  Branches     1458     1460    +2     
=======================================
+ Hits         6673     6675    +2     
  Misses        626      626           

Files	Coverage Δ
...lemetry-instrumentation-koa/src/instrumentation.ts	97.53% <100.00%> (ø)
...ode/opentelemetry-instrumentation-koa/src/utils.ts	100.00% <100.00%> (ø)

[ramesius]

@blumamir Sure, just pushed up a test for the functionality 👍🏻

[ramesius]

@david-luna Probably makes sense. I can raise a seperate PR with that feedback if you don't mind.

[blumamir]

Thanks! 🥇

[ramesius]

CI was failing due to linting, just fixed that up 🤞🏻