Sense: trailing space after URL shouldn't break request parsing

Closes elastic#383

sense/app/sense_editor/editor.js

@@ -239,6 +239,10 @@ define([
           request.url += t.value;
           t = tokenIter.stepForward();
         }
+        if (editor.parser.isEmptyToken(t)) {
+          // if the url row ends with some spaces, skip them.
+          t = editor.parser.nextNonEmptyToken(tokenIter);
+        }
 
         var bodyStartRow = (t ? 0 : 1) + tokenIter.getCurrentTokenRow(); // artificially increase end of docs.
         var bodyStartColumn = 0;
@@ -297,7 +301,7 @@ define([
         }
       }
 
-      var column = (session.getLine(curRow) || "").length;
+      var column = (session.getLine(curRow) || "").replace(/\s+$/, "").length;
 
       return { row: curRow, column: column};
     };

sense/tests/src/editor_tests.js

@@ -131,6 +131,30 @@ define([
     });
   });
 
+  utils_test("simple request range, suffixed with spaces", simple_request.prefix + "   ", simple_request.data + "  ", function () {
+    input.getCurrentRequestRange(function (range) {
+      var expected = new aceRange.Range(
+        0, 0,
+        3, 1
+      );
+      deepEqual(range, expected);
+      start();
+    });
+  });
+
+  utils_test("simple request data, suffixed with spaces", simple_request.prefix + "    " , simple_request.data + " ", function () {
+    input.getCurrentRequest(function (request) {
+      var expected = {
+        method: "POST",
+        url: "_search",
+        data: [simple_request.data]
+      };
+
+      deepEqual(request, expected);
+      start();
+    });
+  });
+
 
   utils_test("single line request range", single_line_request.prefix, single_line_request.data, function () {
     input.getCurrentRequestRange(function (range) {