Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code

Unified Diff: lib/whitelisting.js

Issue 4544964214128640: Issue 1841 - Replaced URI class with built-in URL objects (Closed)
Patch Set: Created Jan. 21, 2015, 7:54 a.m.
Use n/p to move between diff chunks; N/P to move between comments.
Jump to:
View side-by-side diff with in-line comments
Download patch
« lib/url.js ('K') | « lib/url.js ('k') | metadata.common » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments 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
Wladimir Palant 2015/01/21 15:33:44 I was trying to figure out whether you are using a
+ window.navigator.userAgent // HTTP_USER_AGENT
];
return verifySignature(key, signature, params.join("\0"));
« lib/url.js ('K') | « lib/url.js ('k') | metadata.common » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld