refactor(auth): refactor get url params

app/portainer/helpers/urlHelper.js

@@ -1,21 +1,24 @@
-angular.module('portainer.app').service('urlHelper', function urlHelper($location) {
+angular.module('portainer.app').service('urlHelper', function urlHelper($window) {
-
   this.getParameter = getParameter;
+  this.cleanParameters = cleanParameters;
 
   function getParameter(param) {
-    var url = $location.absUrl();
+    var parameters = extractParameters();
-    var index = url.indexOf('?');
+    return parameters[param];
-    if (index < 0) {
-      return;
   }
-    var params = url.substring(index + 1);
+
-    params = params.split('&');
+  function extractParameters() {
-    for (var i = 0; i < params.length; i++) {
+    var queryString = $window.location.search.replace(/.*?\?/,'').split('&');
-      var parameter = params[i].split('=');
+    return queryString.reduce(function(acc, keyValStr) {
-        if (parameter[0] === param) {
+      var keyVal = keyValStr.split('=');
-          return parameter[1].split('#')[0];
+      var key = keyVal[0];
+      var val = keyVal[1];
+      acc[key] = val;
+      return acc;
+    }, {});
   }
-    }
+
-    return;
+  function cleanParameters() {
+    $window.location.search = '';
   }
 });

app/portainer/views/auth/authController.js

@@ -114,8 +114,9 @@ function (urlHelper, $q, $scope, $state, $stateParams, $sanitize, Authentication
   }
 
   function oAuthLogin(code) {
-    Authentication.oAuthLogin(code)
+    return Authentication.oAuthLogin(code)
     .then(function success() {
+      urlHelper.cleanParameters();
       $state.go('portainer.home');
     })
     .catch(function error() {