Skip to content

Commit 5e46559

Browse files
Manifest V3 update. (#80)
* Updates for Manifest V3 - manifest version updated to 3 - remove webRequestBlocking permissions - background scripts are now service_workers - browser_action has been rolled into generic action - fix keyboard shortcut per #80 (comment) from @khlr's * Set Add-on ID in manifest * Use both service_worker and scripts to be able to support both Firefox and Chrome. Setting both service_worker and scripts is supported in Chrome 121+ and Firefox 121+. --------- Co-authored-by: Tim van Dijen <tvdijen@gmail.com>
1 parent e6ade7e commit 5e46559

4 files changed

Lines changed: 19 additions & 11 deletions

File tree

bin/build.sh

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -60,7 +60,7 @@ git archive --format tar "$RELEASE" | (cd "$DEST"; tar xv)
6060

6161
cd "$DEST"
6262

63-
zip -r "$TARGET" *
63+
zip -1 -r "$TARGET" *
6464

6565
cd /
6666
rm -rf "$DEST"

bootstrap.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -3,7 +3,7 @@
33
// The onOpenWindow event handler was slightly modified to be compatible with standard Firefox.
44

55
var browser = browser || chrome
6-
browser.browserAction.onClicked.addListener((tab) => showTracerWindow());
6+
browser.action.onClicked.addListener((tab) => showTracerWindow());
77

88
var tracerWindow = null;
99

manifest.json

Lines changed: 14 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -1,8 +1,8 @@
11
{
22
"name": "SAML-tracer",
33
"description": "A debugger for viewing SAML messages",
4-
"author": "Olav Morken, Jaime Perez, Thijs Kinkhorst, Jan Köhler",
5-
"manifest_version": 2,
4+
"author": "Olav Morken, Jaime Perez, Thijs Kinkhorst, Jan Köhler, Tim van Dijen",
5+
"manifest_version": 3,
66
"version": "1.8.0",
77
"homepage_url": "https://github.com/SimpleSAMLphp/SAML-tracer",
88
"icons": {
@@ -13,14 +13,16 @@
1313
"128": "src/resources/images/icon128.png"
1414
},
1515
"permissions": [
16-
"webRequest",
17-
"webRequestBlocking",
16+
"webRequest"
17+
],
18+
"host_permissions": [
1819
"<all_urls>"
1920
],
2021
"background": {
22+
"service_worker": "bootstrap.js",
2123
"scripts": ["bootstrap.js"]
2224
},
23-
"browser_action": {
25+
"action": {
2426
"default_icon": {
2527
"16": "src/resources/images/icon16.png",
2628
"32": "src/resources/images/icon32.png",
@@ -29,8 +31,14 @@
2931
"128": "src/resources/images/icon128.png"
3032
}
3133
},
34+
"browser_specific_settings": {
35+
"gecko": {
36+
"id": "{d3e01ff2-9a3a-4007-8f6e-7acd9a5de263}",
37+
"strict_min_version": "121.0"
38+
}
39+
},
3240
"commands": {
33-
"_execute_browser_action": {
41+
"_execute_action": {
3442
"suggested_key": {
3543
"default": "Alt+Shift+S"
3644
},

src/SAMLTrace.js

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -905,19 +905,19 @@ SAMLTrace.TraceWindow.init = function() {
905905
browser.webRequest.onBeforeRequest.addListener(
906906
traceWindow.saveNewRequest,
907907
{urls: ["<all_urls>"]},
908-
["blocking", "requestBody"]
908+
["requestBody"]
909909
);
910910

911911
browser.webRequest.onBeforeSendHeaders.addListener(
912912
traceWindow.attachHeadersToRequest,
913913
{urls: ["<all_urls>"]},
914-
['blocking', 'requestHeaders', browser.webRequest.OnBeforeSendHeadersOptions.EXTRA_HEADERS].filter(Boolean)
914+
['requestHeaders', browser.webRequest.OnBeforeSendHeadersOptions.EXTRA_HEADERS].filter(Boolean)
915915
);
916916

917917
browser.webRequest.onHeadersReceived.addListener(
918918
traceWindow.attachResponseToRequest,
919919
{urls: ["<all_urls>"]},
920-
['blocking', 'responseHeaders', browser.webRequest.OnHeadersReceivedOptions.EXTRA_HEADERS].filter(Boolean)
920+
['responseHeaders', browser.webRequest.OnHeadersReceivedOptions.EXTRA_HEADERS].filter(Boolean)
921921
);
922922
};
923923

0 commit comments

Comments
 (0)