#1303 Add MFA disable confirm in user settings page

TheHive-Project · May 2, 2020 · 5187440 · 5187440
1 parent 15c80dd
commit 5187440
Showing 1 changed file with 17 additions and 5 deletions.
diff --git a/frontend/app/scripts/controllers/SettingsCtrl.js b/frontend/app/scripts/controllers/SettingsCtrl.js
@@ -1,7 +1,7 @@
 (function() {
     'use strict';
     angular.module('theHiveControllers').controller('SettingsCtrl',
-        function($scope, $state, UserSrv, AuthenticationSrv, NotificationSrv, clipboard, resizeService, readLocalPicService, currentUser, appConfig) {
+        function($scope, $state, UserSrv, ModalSrv, AuthenticationSrv, NotificationSrv, clipboard, resizeService, readLocalPicService, currentUser, appConfig) {
             $scope.currentUser = currentUser;
             $scope.appConfig = appConfig;
 
@@ -126,14 +126,26 @@
             };
 
             $scope.resetMfa = function() {
-                UserSrv.resetMfa()
-                    .then(function(/*response*/) {                        
+                var modalInstance = ModalSrv.confirm(
+                    'Disable MFA',
+                    'Are you sure you want to disabble MFA settings?', {
+                        okText: 'Yes, disable it',
+                        flavor: 'danger'
+                    }
+                );
+
+                modalInstance.result
+                    .then(function() {
+                        UserSrv.resetMfa();
+                    })
+                    .then(function() {
                         NotificationSrv.log('Your multi-factor authentication has been successfully disabled', 'success');
                         $state.reload();
-
                     })
                     .catch(function(err) {
-                        NotificationSrv.error('SettingsCtrl', err.data, err.status);
+                        if (!_.isString(err)) {
+                            NotificationSrv.error('SettingsCtrl', err.data, err.status);
+                        }
                     });
             };