Changeset 474 for TFP-WebServer/WebServer/src/Permissions/AdminApiTokens.cs

Timestamp:

Aug 24, 2023, 12:48:17 PM (15 months ago)

Author:

alloc

Message:

Added locks on AdminTools instead of local to the individual permission systems

File:

• TFP-WebServer/WebServer/src/Permissions/AdminApiTokens.cs (modified) (2 diffs)

TFP-WebServer/WebServer/src/Permissions/AdminApiTokens.cs

@@ -40 +40 @@
 
                 public void AddToken (string _name, string _secret, int _permissionLevel) {
-                        ApiToken apiToken = new ApiToken (_name, _secret, _permissionLevel);
-                        tokens [_name] = apiToken;
-                        Parent.Save ();
+                        lock (Parent) {
+                                ApiToken apiToken = new ApiToken (_name, _secret, _permissionLevel);
+                                tokens[_name] = apiToken;
+                                Parent.Save ();
+                        }
                 }
 
                 public bool RemoveToken (string _name) {
-                        bool removed = tokens.Remove (_name);
-                        if (removed) {
-                                Parent.Save ();
+                        lock (Parent) {
+                                bool removed = tokens.Remove (_name);
+                                if (removed) {
+                                        Parent.Save ();
+                                }
+
+                                return removed;
                         }
-
-                        return removed;
                 }
 
                 public Dictionary<string, ApiToken> GetTokens () {
-                        return tokens;
+                        lock (Parent) {
+                                return tokens;
+                        }
                 }
 
@@ -113 +119 @@
 
                 public int GetPermissionLevel (string _name, string _secret) {
-                        if (tokens.TryGetValue (_name, out ApiToken apiToken) && apiToken.Secret == _secret) {
-                                return apiToken.PermissionLevel;
+                        lock (Parent) {
+                                if (tokens.TryGetValue (_name, out ApiToken apiToken) && apiToken.Secret == _secret) {
+                                        return apiToken.PermissionLevel;
+                                }
+
+                                if (IsCommandlineToken (_name, _secret)) {
+                                        return 0;
+                                }
+
+                                return int.MaxValue;
                         }
-
-                        if (IsCommandlineToken (_name, _secret)) {
-                                return 0;
-                        }
-
-                        return int.MaxValue;
                 }