fix(core): support regex global flag in urlMatches

open-telemetry · Oct 22, 2021 · 15b5c59 · 15b5c59
1 parent 6a8c2f2
commit 15b5c59
Show file tree

Hide file tree

Showing 2 changed files with 13 additions and 1 deletion.
diff --git a/packages/opentelemetry-core/src/utils/url.ts b/packages/opentelemetry-core/src/utils/url.ts
@@ -17,7 +17,7 @@ export function urlMatches(url: string, urlToMatch: string | RegExp): boolean {
   if (typeof urlToMatch === 'string') {
     return url === urlToMatch;
   } else {
-    return urlToMatch.test(url);
+    return !!url.match(urlToMatch);
   }
 }
 /**

diff --git a/packages/opentelemetry-core/test/utils/url.test.ts b/packages/opentelemetry-core/test/utils/url.test.ts
@@ -71,5 +71,17 @@ describe('Core - Utils - url', () => {
         );
       });
     });
+    describe('when regex has global flag', () => {
+      it('should return true', () => {
+        const ignoredUrls = [/myaddr/g];
+        for (let i = 0; i < 2; i++) {
+          assert.strictEqual(
+            isUrlIgnored(urlToTest, ignoredUrls),
+            true,
+            urlIgnored
+          );
+        }
+      });
+    });
   });
 });