AG-36463 Fix filtering log of $permissions

Merge in ADGUARD-FILTERS/tsurlfilter from fix/AG-36463 to master Squashed commit of the following: commit 7c1c7fb Author: Kurbanali Ruslan <[email protected]> Date: Thu Oct 10 19:13:38 2024 +0500 removed requestId from permissions event commit 6e21173 Author: Kurbanali Ruslan <[email protected]> Date: Thu Oct 10 14:22:20 2024 +0500 added requestId to permission filter log event commit b490c41 Author: Kurbanali Ruslan <[email protected]> Date: Thu Oct 10 13:17:38 2024 +0500 added comment commit 2ba8f52 Author: Kurbanali Ruslan <[email protected]> Date: Thu Oct 10 13:17:12 2024 +0500 mirroring fix
AdguardTeam · Oct 10, 2024 · 7383903 · 7383903
1 parent dfed614
commit 7383903
Showing 1 changed file with 5 additions and 1 deletion.
diff --git a/packages/tswebextension/src/lib/mv2/background/services/permissions-policy-service.ts b/packages/tswebextension/src/lib/mv2/background/services/permissions-policy-service.ts
@@ -5,6 +5,7 @@ import {
     PERMISSIONS_POLICY_HEADER_NAME,
     RequestType,
 } from '@adguard/tsurlfilter';
+import { nanoid } from 'nanoid';
 import {
     defaultFilteringLog,
     FilteringEventType,
@@ -108,7 +109,10 @@ export class PermissionsPolicyService {
                     type: FilteringEventType.ApplyPermissionsRule,
                     data: {
                         tabId,
-                        eventId: requestId,
+                        // for proper filtering log request info rule displaying
+                        // event id should be unique for each event, not copied from request
+                        // https://github.com/AdguardTeam/AdguardBrowserExtension/issues/2341
+                        eventId: nanoid(),
                         requestUrl,
                         frameUrl: referrerUrl,
                         frameDomain: getDomain(referrerUrl),