Update function comment and replace URL usage with regex.

Author: zijianjoy

packages/performance/src/services/perf_logger.ts

@@ -153,13 +153,12 @@ export function logNetworkRequest(networkRequest: NetworkRequest): void {
 
   // Do not log the js sdk's call to transport service domain to avoid unnecessary cycle.
   // Need to blacklist both old and new endpoints to avoid migration gap.
-  const networkRequestHostName = new URL(networkRequest.url).hostname;
+  const networkRequestHostName = extractDomain(networkRequest.url);
 
   // Blacklist old log endpoint and new transport endpoint.
   // Because Performance SDK doesn't instrument requests sent from SDK itself.
-  const logEndpointHostName = new URL(settingsService.logEndPointUrl).hostname;
-  const flEndpointHostName = new URL(settingsService.flTransportEndpointUrl)
-    .hostname;
+  const logEndpointHostName = extractDomain(settingsService.logEndPointUrl);
+  const flEndpointHostName = extractDomain(settingsService.flTransportEndpointUrl);
   if (
     networkRequestHostName === logEndpointHostName ||
     networkRequestHostName === flEndpointHostName
@@ -177,6 +176,15 @@ export function logNetworkRequest(networkRequest: NetworkRequest): void {
   setTimeout(() => sendLog(networkRequest, ResourceType.NetworkRequest), 0);
 }
 
+function extractDomain(url: string): string {
+  const urlRegex = /(https?:\/\/)?([-a-zA-Z0-9@:%._\+~#=]{2,256}\.[a-z]{2,4})\b([-a-zA-Z0-9@:%_\+.~#?&//=]*)/g;
+  let matched = urlRegex.exec(url);
+  if (matched && matched[1]) {
+    return matched[1];
+  }
+  return "";
+}
+
 function serializer(
   resource: NetworkRequest | Trace,
   resourceType: ResourceType

packages/performance/src/services/remote_config_service.ts

@@ -271,9 +271,6 @@ function shouldLogAfterSampling(samplingRate: number): boolean {
 /**
  * True if event should be sent to Fl transport endpoint rather than CC transport endpoint.
  * rolloutPercent is in range [0.0, 100.0].
- * @param {string} iid Installation ID which identifies a web app installed on client.
- * @param {number} rolloutPercent the possibility of this app sending events to Fl endpoint.
- * @return {boolean} true if this installation should send events to Fl endpoint.
  */
 export function isDestFl(iid: string, rolloutPercent: number): boolean {
   if (iid.length === 0) {
@@ -283,8 +280,6 @@ export function isDestFl(iid: string, rolloutPercent: number): boolean {
 }
 /**
  * Generate integer value range in [0, 99]. Implementation from String.hashCode() in Java.
- * @param {string} seed Same seed will generate consistent hash value using this algorithm.
- * @return {number} Hash value in range [0, 99], generated from seed and hash algorithm.
  */
 function getHashPercent(seed: string): number {
   let hash = 0;