From 280d3c42d9d519d381b8bae64e1dfc7e2cf42a2f Mon Sep 17 00:00:00 2001 From: Wojtek Kosior Date: Mon, 3 Jan 2022 22:30:52 +0100 Subject: improve and test the dafult policy dialog This commit also fixes some bugs that manifested themselves spuriously. --- background/broadcast_broker.js | 3 ++- background/webrequest.js | 2 +- 2 files changed, 3 insertions(+), 2 deletions(-) (limited to 'background') diff --git a/background/broadcast_broker.js b/background/broadcast_broker.js index 5af3a7b..458fb7f 100644 --- a/background/broadcast_broker.js +++ b/background/broadcast_broker.js @@ -54,7 +54,8 @@ function new_broadcast_listener(port) { const listener_ctx = {port, id: ++next_id, channels: new Set()}; port.onMessage.addListener(msg => listener_command(msg, listener_ctx)); - port.onDisconnect.addListener(msg => listener_remove(msg, listener_ctx)); + const on_disconnect = () => remove_broadcast_listener(listener_ctx); + port.onDisconnect.addListener(on_disconnect); } function listener_command(msg, listener_ctx) diff --git a/background/webrequest.js b/background/webrequest.js index e32947a..9787318 100644 --- a/background/webrequest.js +++ b/background/webrequest.js @@ -74,7 +74,7 @@ function on_headers_received(details) let headers = details.responseHeaders; - const policy = decide_policy(tree, details.url, default_allow, secret); + const policy = decide_policy(tree, details.url, !!default_allow, secret); if (policy.allow) return; -- cgit v1.2.3