Trim URLbar input

Fix #519 Auditors: @diracdeltas
brave · Feb 1, 2016 · 22a6ea3 · 22a6ea3 · diracdeltas · Feb 2, 2016
1 parent 559f9aa
commit 22a6ea3
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 0 deletions.
diff --git a/js/actions/windowActions.js b/js/actions/windowActions.js
@@ -42,6 +42,7 @@ const WindowActions = {
    * @param {string} location - The URL of the page to load
    */
   loadUrl: function (activeFrame, location) {
+    location = location.trim()
     let newFrame = false
     if (activeFrame.get('isPinned')) {
       try {
@@ -82,6 +83,7 @@ const WindowActions = {
    * it is active the URL text will also be changed.
    */
   setLocation: function (location, key) {
+    location = location.trim()
     if (UrlUtil.isURL(location)) {
       location = UrlUtil.getUrlFromInput(location)
     }

diff --git a/js/lib/appUrlUtil.js b/js/lib/appUrlUtil.js
@@ -83,5 +83,6 @@ export function isPrivilegedUrl (input) {
  * @param {string} input
  */
 export function isUrl (input) {
+  input = input.trim()
   return (UrlUtil.isURL(input) && !input.includes(' '))
 }