feat: allow setting attributes when recording exceptions

[vreynolds]

Which problem is this PR solving?

• span.recordException needs to allow setting event attributes per spec

If RecordException is provided, the method MUST accept an optional parameter to provide any additional event attributes (this SHOULD be done in the same way as for the AddEvent method). If attributes with the same name would be generated by the method already, the additional attributes take precedence.

• API change: feat: allow setting attributes when recording exceptions opentelemetry-js-api#97

Short description of the changes

• allow setting attributes when recording exception, same as the span.addEvent interface

[codecov]

Codecov Report

Merging #2298 (5665c42) into main (7860344) will increase coverage by 0.06%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main    #2298      +/-   ##
==========================================
+ Coverage   92.68%   92.75%   +0.06%     
==========================================
  Files         145      145              
  Lines        5184     5188       +4     
  Branches     1060     1058       -2     
==========================================
+ Hits         4805     4812       +7     
+ Misses        379      376       -3     

Impacted Files	Coverage Δ
packages/opentelemetry-tracing/src/Span.ts	98.18% <100.00%> (+0.10%)	⬆️
...s/opentelemetry-instrumentation-fetch/src/fetch.ts	96.91% <0.00%> (-0.08%)	⬇️
...emetry-instrumentation-xml-http-request/src/xhr.ts	97.58% <0.00%> (+0.99%)	⬆️
...emetry-api-metrics/src/platform/node/globalThis.ts	100.00% <0.00%> (+100.00%)	⬆️

[vreynolds]

open-telemetry/opentelemetry-js-api#97 (comment)

[longility]

@vreynolds Would we still want this without the otel api change even though we can't use it so that in the future, we can slot this in for v2 more easily?

[dyladan]

I think we don't need to wait for v2. This feature could be added in a 1.x minor release.

[longility]

Even better