prevent users deleting the only admin user docker-homebridge#83

CHANGELOG.md

@@ -7,6 +7,7 @@ All notable changes to this project will be documented in this file. This projec
 ### Bug Fixes
 
 * Prevent zoom when using the config editor on iOS (@Yanni via Homebridge Slack)
+* Prevent the deletion of an admin user if there are no other admin users ([docker-homebridge#83](https://github.com/oznu/docker-homebridge/issues/83))
 
 ## 3.3.0 (2018-03-16)
 

src/users.ts

@@ -121,6 +121,11 @@ class Users {
       throw new Error('User not found');
     }
 
+    // prevent deleting the only admin user
+    if (authfile[index].admin && authfile.filter(x => x.admin === true).length < 2) {
+      throw new Error('Cannot delete only admin user');
+    }
+
     authfile.splice(index, 1);
 
     // update the auth.json

ui/src/app/users/users.component.ts

@@ -45,7 +45,7 @@ export class UsersComponent implements OnInit {
         this.$state.reload();
       },
       err => {
-        this.toastr.error(`Failed To Delete User`, 'Error');
+        this.toastr.error(err.error.message || 'Failed to delete user', 'Error');
       }
     );
   }