canmove, Confirmed users
637
edits
Line 139: | Line 139: | ||
== Review comments == | == Review comments == | ||
* need to whitelist scheme? either a short static list, or at least checkLoadURI? | |||
* javascript: in particular would be bad (but seems to be blocked?) | |||
* what's the principal used for data: urls? | |||
* what referrer gets sent? | |||
* Jesse thinks DNS prefetching needs to be disabled for privacy since we might linkify things that weren't actual links in the page. (<foo href="http://blah.com"> will get linkified). | |||
* make sure URI's added can't be used for script injection, extra quotes don't mess it up, etc. |