Unified Diff: lib/whitelisting.js
Issue 5006868082589696:
Noissue - Removed unused docDomain argument of verifyKey() (Closed)
Patch Set:
Created Jan. 21, 2015, 7:57 a.m.
Use n/p to move between diff chunks;
N/P to move between comments.
« no previous file with comments
|
« no previous file
|
no next file »
|
no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Hide Comments ('s')
Expand Comments ('e') | Collapse Comments ('c') | Hide Comments ('s')
| Index: lib/whitelisting.js |
| =================================================================== |
| --- a/lib/whitelisting.js |
| +++ b/lib/whitelisting.js |
| @@ -60,7 +60,7 @@ |
| return null; |
| } |
| -let verifyKey = function(key, signature, url, docDomain) |
| +let verifyKey = function(key, signature, url) |
| { |
| url = new URL(url); |
| let params = [ |
| @@ -88,13 +88,13 @@ |
| let processKey = exports.processKey = function(token, page, frame) |
| { |
| let url = stripFragmentFromURL(frame.url); |
| - let docDomain = extractHostFromURL((frame.parent || frame).url); |
| if (token.indexOf("_") < 0) |
| return; |
| let [key, signature] = token.split("_", 2); |
| key = key.replace(/=/g, ""); |
| - if (verifyKey(key, signature, url, docDomain)) |
| + |
| + if (verifyKey(key, signature, url)) |
| recordKey(page, url, key); |
| }; |
« no previous file with comments
|
« no previous file
|
no next file »
|
no next file with comments »
