Unified Diff: lib/whitelisting.js
Issue 4544964214128640:
Issue 1841 - Replaced URI class with built-in URL objects (Closed)
Patch Set: Addressed comments
Created Jan. 21, 2015, 5:47 p.m.
Use n/p to move between diff chunks;
N/P to move between comments.
« no previous file with comments
|
« lib/url.js ('k')
|
metadata.common » ('j')
|
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 |
| @@ -62,11 +62,11 @@ |
| let verifyKey = function(key, signature, url, docDomain) |
| { |
| - let uri = new URI(url); |
| + url = new URL(url); |
| let params = [ |
| - uri.path, // REQUEST_URI |
| - uri.asciiHost + (uri.port != -1 ? ":" + uri.port : ""), // HTTP_HOST |
| - window.navigator.userAgent // HTTP_USER_AGENT |
| + url.pathname + url.search, // REQUEST_URI |
| + url.host, // HTTP_HOST |
| + window.navigator.userAgent // HTTP_USER_AGENT |
| ]; |
| return verifySignature(key, signature, params.join("\0")); |
« no previous file with comments
|
« lib/url.js ('k')
|
metadata.common » ('j')
|
no next file with comments »
