me/omegafox
1
0
Fork 0
mirror of https://forge.fsky.io/oneflux/omegafox.git synced 2026-02-10 21:32:05 -08:00
Code Issues Projects Releases Packages Wiki Activity Actions

Experimental observer leak fix

Browse source 
Juggler will no longer notify content-document-global-created.
This patch also reverses the _Runtime domain rename fix due to Playwright breakage.

This was causing leaks with some WAFs.
This commit is contained in:
daijro 2024-08-13 06:24:53 -05:00
parent b5d16a902f
commit 2f57040710
3 changed files with 2 additions and 3 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

2
README.md
View file

@ -264,7 +264,7 @@ Miscellaneous (WebGl spoofing, battery status, etc)
- Custom implementation of Playwright for the latest Firefox - Custom implementation of Playwright for the latest Firefox
- Various config patches to evade bot detection - Various config patches to evade bot detection
- Removed leaking Playwright patches: - Removed leaking Playwright patches:
- Fixes `Runtime` domain detection - Fixes `content-document-global-created` observer leak
- Fixes `navigator.webdriver` detection - Fixes `navigator.webdriver` detection
- Removed potentially leaking anti-zoom/meta viewport handling patches - Removed potentially leaking anti-zoom/meta viewport handling patches
- Re-enable fission content isolation - Re-enable fission content isolation

1
additions/juggler/content/FrameTree.js
View file

@ -68,7 +68,6 @@ class FrameTree {
frame._pendingNavigationId = helper.toProtocolNavigationId(loadIdentifier); frame._pendingNavigationId = helper.toProtocolNavigationId(loadIdentifier);
this.emit(FrameTree.Events.NavigationStarted, frame); this.emit(FrameTree.Events.NavigationStarted, frame);
}, 'juggler-navigation-started-renderer'), }, 'juggler-navigation-started-renderer'),
helper.addObserver(this._onDOMWindowCreated.bind(this), 'content-document-global-created'),
helper.addObserver(this._onDOMWindowCreated.bind(this), 'juggler-dom-window-reused'), helper.addObserver(this._onDOMWindowCreated.bind(this), 'juggler-dom-window-reused'),
helper.addObserver((browsingContext, topic, why) => { helper.addObserver((browsingContext, topic, why) => {
this._onBrowsingContextAttached(browsingContext); this._onBrowsingContextAttached(browsingContext);

2
additions/juggler/protocol/Protocol.js
View file

@ -577,7 +577,7 @@ const Network = {
}, },
}; };
const _Runtime = { const Runtime = {
targets: ['page'], targets: ['page'],
types: runtimeTypes, types: runtimeTypes,
events: { events: {