feat: record exceptions in http instrumentation

experimental/packages/opentelemetry-instrumentation-http/src/utils.ts

@@ -150,6 +150,7 @@ export const setSpanWithError = (
   });
 
   span.setStatus({ code: SpanStatusCode.ERROR, message });
+  span.recordException(error);
 };
 
 /**

experimental/packages/opentelemetry-instrumentation-http/test/functionals/http-enable.test.ts

@@ -570,6 +570,7 @@ describe('HttpInstrumentation', () => {
             },
             component: 'http',
             noNetPeer: true,
+            error: err,
           };
           assertSpan(spans[0], SpanKind.CLIENT, validations);
           return true;

experimental/packages/opentelemetry-instrumentation-http/test/utils/assertSpan.ts

@@ -13,7 +13,7 @@
  * See the License for the specific language governing permissions and
  * limitations under the License.
  */
-import { isValidSpanId, SpanKind, SpanStatus } from '@opentelemetry/api';
+import { isValidSpanId, SpanKind, SpanStatus, Exception } from '@opentelemetry/api';
 import { hrTimeToNanoseconds } from '@opentelemetry/core';
 import { ReadableSpan } from '@opentelemetry/sdk-trace-base';
 import { SemanticAttributes } from '@opentelemetry/semantic-conventions';
@@ -38,6 +38,7 @@ export const assertSpan = (
     serverName?: string;
     component: string;
     noNetPeer?: boolean; // we don't expect net peer info when request throw before being sent
+    error?: Exception;
   }
 ) => {
   assert.strictEqual(span.spanContext().traceId.length, 32);
@@ -65,7 +66,13 @@ export const assertSpan = (
   );
 
   assert.strictEqual(span.links.length, 0);
-  assert.strictEqual(span.events.length, 0);
+
+  if (validations.error) {
+    assert.strictEqual(span.events.length, 1);
+    assert.strictEqual(span.events[0].name, 'exception');
+  } else {
+    assert.strictEqual(span.events.length, 0);
+  }
 
   assert.deepStrictEqual(
     span.status,