#1317 Hide mfa related columns in user admin if auth.multifactor.enab…

…led = false
TheHive-Project · May 24, 2020 · 636d89f · 636d89f
1 parent 465041f
commit 636d89f
Show file tree

Hide file tree

Showing 5 changed files with 9 additions and 4 deletions.
diff --git a/frontend/app/scripts/app.js b/frontend/app/scripts/app.js
@@ -237,6 +237,9 @@ angular.module('thehive', [
                     },
                     fields: function(CustomFieldsSrv){
                         return CustomFieldsSrv.all();
+                    },
+                    appConfig: function(VersionSrv) {
+                        return VersionSrv.get();
                     }
                 },
                 permissions: ['manageOrganisation', 'manageUser', 'manageCaseTemplate']

diff --git a/frontend/app/scripts/components/organisation/OrgUserListCmp.js b/frontend/app/scripts/components/organisation/OrgUserListCmp.js
@@ -202,6 +202,7 @@
             templateUrl: 'views/components/org/user.list.html',
             bindings: {
                 users: '<',
+                mfaEnabled: '<',
                 onReload: '&',
                 onEdit: '&'
             }

diff --git a/frontend/app/scripts/controllers/admin/organisation/OrgDetailsCtrl.js b/frontend/app/scripts/controllers/admin/organisation/OrgDetailsCtrl.js
@@ -2,13 +2,14 @@
     'use strict';
 
     angular.module('theHiveControllers').controller('OrgDetailsCtrl',
-        function($scope, $q, $uibModal, OrganisationSrv, NotificationSrv, UserSrv, organisation, users, templates, fields) {
+        function($scope, $q, $uibModal, OrganisationSrv, NotificationSrv, UserSrv, organisation, users, templates, fields, appConfig) {
             var self = this;
 
             this.org = organisation;
             this.users = users;
             this.templates = templates;
             this.fields = fields;
+            this.canChangeMfa = appConfig.config.capabilities.indexOf('mfa') !== -1;
 
             this.getUserInfo = UserSrv.getCache;
 

diff --git a/frontend/app/views/components/org/user.list.html b/frontend/app/views/components/org/user.list.html
@@ -18,7 +18,7 @@
                     <th></th>
                     <th>Password</th>
                     <th>API Key</th>
-                    <th class="text-center" style="width: 100px">MFA</th>
+                    <th class="text-center" style="width: 100px" ng-if="$ctrl.mfaEnabled">MFA</th>
                     <th style="width: 180px"></th>
                 </tr>
             </thead>
@@ -84,7 +84,7 @@ <h4 class="media-heading">
                             </div>
                         </span>
                     </td>
-                    <td class="text-center">
+                    <td class="text-center" ng-if="$ctrl.mfaEnabled">
                         <span ng-if="!user.hasMFA">
                             No
                         </span>

diff --git a/frontend/app/views/partials/admin/organisation/details.html b/frontend/app/views/partials/admin/organisation/details.html
@@ -65,7 +65,7 @@ <h3 class="box-title">
                                     </div>
                                 </div>
                             </div>
-                            <org-user-list users="$vm.users" on-reload="$vm.reloadUsers()" on-edit="$vm.showUserDialog(user)"></org-user-list>
+                            <org-user-list users="$vm.users" on-reload="$vm.reloadUsers()" on-edit="$vm.showUserDialog(user)" mfa-enabled="$vm.canChangeMfa"></org-user-list>
                         </div>
                     </uib-tab>
                     <uib-tab index="1" if-permission="manageCaseTemplate">