Merge pull request prose#1014 from prose/feature/1013-url-query-parsing

Don't use regex to parse url

Author: dereklieu

app/models/user.js

@@ -1,5 +1,6 @@
 var $ = require('jquery-browserify');
 var _ = require('underscore');
+var url = require('url');
 
 var Backbone = require('backbone');
 var Repos = require('../collections/repos');
@@ -19,21 +20,23 @@ module.exports = Backbone.Model.extend({
   },
 
   authenticate: function(options) {
-    var match;
-
     if (cookie.get('oauth-token')) {
       if (_.isFunction(options.success)) options.success();
     } else {
-      match = window.location.href.match(/\?code=([a-z0-9]*)/);
-
-      if (match) {
-        var ajax = $.ajax(auth.url + '/authenticate/' + match[1], {
+      var parsed = url.parse(window.location.href, true);
+      var code = parsed.query && parsed.query.code;
+      if (code) {
+        var ajax = $.ajax(auth.url + '/authenticate/' + code, {
           success: function(data) {
             cookie.set('oauth-token', data.token);
-
-            var regex = new RegExp("(?:\\/)?\\?code=" + match[1]);
-            window.location.href = window.location.href.replace(regex, '');
-
+            var newHref = url.format({
+              protocol: parsed.protocol,
+              slashes: parsed.slashes,
+              host: parsed.host,
+              pathname: parsed.pathname,
+              hash: parsed.hash
+            });
+            window.location.href = newHref;
             if (_.isFunction(options.success)) options.success();
           }
         });