fix(instrumentation-http)!: verbose log if URL passed to http request…

… cannot be parsed

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

@@ -762,6 +762,7 @@ export class HttpInstrumentation extends InstrumentationBase<HttpInstrumentation
           ? (args.shift() as http.RequestOptions)
           : undefined;
       const { method, invalidUrl, optionsParsed } = getRequestInfo(
+        instrumentation._diag,
         options,
         extraOptions
       );

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

@@ -297,10 +297,12 @@ function stringUrlToHttpOptions(
 /**
  * Makes sure options is an url object
  * return an object with default value and parsed options
+ * @param logger component logger
  * @param options original options for the request
  * @param [extraOptions] additional options for the request
  */
 export const getRequestInfo = (
+  logger: DiagLogger,
   options: url.URL | RequestOptions | string,
   extraOptions?: RequestOptions
 ): {
@@ -321,6 +323,10 @@ export const getRequestInfo = (
       pathname = convertedOptions.pathname || '/';
     } catch (e) {
       invalidUrl = true;
+      logger.verbose(
+        'Unable to parse URL provided to HTTP request, using fallback to determine path. Original error:',
+        e
+      );
       // for backward compatibility with how url.parse() behaved.
       optionsParsed = {
         path: options,

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

@@ -102,7 +102,7 @@ describe('Utility', () => {
         urlParsedWithUndefinedHostAndNullPort,
         whatWgUrl,
       ]) {
-        const result = utils.getRequestInfo(param);
+        const result = utils.getRequestInfo(diag, param);
         assert.strictEqual(result.optionsParsed.hostname, 'google.fr');
         assert.strictEqual(result.optionsParsed.protocol, 'http:');
         assert.strictEqual(result.optionsParsed.path, '/aPath?qu=ry');