Skip to content

Commit

Permalink
Manifest V3 update. (#80)
Browse files Browse the repository at this point in the history
* 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 <[email protected]>
  • Loading branch information
kellenmurphy and tvdijen authored Aug 17, 2024
1 parent e6ade7e commit 5e46559
Show file tree
Hide file tree
Showing 4 changed files with 19 additions and 11 deletions.
2 changes: 1 addition & 1 deletion bin/build.sh
Original file line number Diff line number Diff line change
Expand Up @@ -60,7 +60,7 @@ git archive --format tar "$RELEASE" | (cd "$DEST"; tar xv)

cd "$DEST"

zip -r "$TARGET" *
zip -1 -r "$TARGET" *

cd /
rm -rf "$DEST"
Expand Down
2 changes: 1 addition & 1 deletion bootstrap.js
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,7 @@
// The onOpenWindow event handler was slightly modified to be compatible with standard Firefox.

var browser = browser || chrome
browser.browserAction.onClicked.addListener((tab) => showTracerWindow());
browser.action.onClicked.addListener((tab) => showTracerWindow());

var tracerWindow = null;

Expand Down
20 changes: 14 additions & 6 deletions manifest.json
Original file line number Diff line number Diff line change
@@ -1,8 +1,8 @@
{
"name": "SAML-tracer",
"description": "A debugger for viewing SAML messages",
"author": "Olav Morken, Jaime Perez, Thijs Kinkhorst, Jan Köhler",
"manifest_version": 2,
"author": "Olav Morken, Jaime Perez, Thijs Kinkhorst, Jan Köhler, Tim van Dijen",
"manifest_version": 3,
"version": "1.8.0",
"homepage_url": "https://github.com/SimpleSAMLphp/SAML-tracer",
"icons": {
Expand All @@ -13,14 +13,16 @@
"128": "src/resources/images/icon128.png"
},
"permissions": [
"webRequest",
"webRequestBlocking",
"webRequest"
],
"host_permissions": [
"<all_urls>"
],
"background": {
"service_worker": "bootstrap.js",
"scripts": ["bootstrap.js"]
},
"browser_action": {
"action": {
"default_icon": {
"16": "src/resources/images/icon16.png",
"32": "src/resources/images/icon32.png",
Expand All @@ -29,8 +31,14 @@
"128": "src/resources/images/icon128.png"
}
},
"browser_specific_settings": {
"gecko": {
"id": "{d3e01ff2-9a3a-4007-8f6e-7acd9a5de263}",
"strict_min_version": "121.0"
}
},
"commands": {
"_execute_browser_action": {
"_execute_action": {
"suggested_key": {
"default": "Alt+Shift+S"
},
Expand Down
6 changes: 3 additions & 3 deletions src/SAMLTrace.js
Original file line number Diff line number Diff line change
Expand Up @@ -905,19 +905,19 @@ SAMLTrace.TraceWindow.init = function() {
browser.webRequest.onBeforeRequest.addListener(
traceWindow.saveNewRequest,
{urls: ["<all_urls>"]},
["blocking", "requestBody"]
["requestBody"]
);

browser.webRequest.onBeforeSendHeaders.addListener(
traceWindow.attachHeadersToRequest,
{urls: ["<all_urls>"]},
['blocking', 'requestHeaders', browser.webRequest.OnBeforeSendHeadersOptions.EXTRA_HEADERS].filter(Boolean)
['requestHeaders', browser.webRequest.OnBeforeSendHeadersOptions.EXTRA_HEADERS].filter(Boolean)
);

browser.webRequest.onHeadersReceived.addListener(
traceWindow.attachResponseToRequest,
{urls: ["<all_urls>"]},
['blocking', 'responseHeaders', browser.webRequest.OnHeadersReceivedOptions.EXTRA_HEADERS].filter(Boolean)
['responseHeaders', browser.webRequest.OnHeadersReceivedOptions.EXTRA_HEADERS].filter(Boolean)
);
};

Expand Down

0 comments on commit 5e46559

Please sign in to comment.