Skip to content

Commit

Permalink
Remove Dependency on InstallTrigger. Replacement is OnBeforeSendHeade… (
Browse files Browse the repository at this point in the history
#75)

* Remove Dependency on InstallTrigger. Replacement is OnBeforeSendHeadersOptions.EXTRA_HEADERS and OnHeadersReceivedOptions.EXTRA_HEADERS predefined constants.

* removed stray spaces
  • Loading branch information
kellenmurphy committed Jul 1, 2022
1 parent a500371 commit c14736c
Showing 1 changed file with 2 additions and 14 deletions.
16 changes: 2 additions & 14 deletions src/SAMLTrace.js
Original file line number Diff line number Diff line change
Expand Up @@ -904,18 +904,6 @@ SAMLTrace.TraceWindow.init = function() {
var browser = browser || chrome;
let traceWindow = new SAMLTrace.TraceWindow();

let isFirefox = () => {
return typeof InstallTrigger !== 'undefined';
}

let getOnBeforeSendHeadersExtraInfoSpec = () => {
return isFirefox() ? ["blocking", "requestHeaders"] : ["blocking", "requestHeaders", "extraHeaders"];
};

let getOnHeadersReceivedExtraInfoSpec = () => {
return isFirefox() ? ["blocking", "responseHeaders"] : ["blocking", "responseHeaders", "extraHeaders"];
};

browser.webRequest.onBeforeRequest.addListener(
traceWindow.saveNewRequest,
{urls: ["<all_urls>"]},
Expand All @@ -925,13 +913,13 @@ SAMLTrace.TraceWindow.init = function() {
browser.webRequest.onBeforeSendHeaders.addListener(
traceWindow.attachHeadersToRequest,
{urls: ["<all_urls>"]},
getOnBeforeSendHeadersExtraInfoSpec()
['blocking', 'requestHeaders', browser.webRequest.OnBeforeSendHeadersOptions.EXTRA_HEADERS].filter(Boolean)
);

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

Expand Down

0 comments on commit c14736c

Please sign in to comment.