Cleaned some code files

This commit is contained in:
Donald Zou
2025-09-15 21:48:00 +08:00
parent 1a26f757a8
commit 3673813e6a
3 changed files with 16 additions and 34 deletions

View File

@@ -791,12 +791,8 @@ def API_addPeers(configName):
break break
if len(keyPairs) == 0 or (bulkAdd and len(keyPairs) != bulkAddAmount): if len(keyPairs) == 0 or (bulkAdd and len(keyPairs) != bulkAddAmount):
return ResponseObject(False, "Generating key pairs by bulk failed") return ResponseObject(False, "Generating key pairs by bulk failed")
status, result = config.addPeers(keyPairs) status, addedPeers, message = config.addPeers(keyPairs)
# DashboardWebHooks.RunWebHook('peer_created', { return ResponseObject(status=status, message=message, data=addedPeers)
# "configuration": config.Name,
# "peers": list(map(lambda p : p['id'], keyPairs))
# })
return ResponseObject(status=status, message=result['message'], data=result['peers'])
else: else:
if config.searchPeer(public_key)[0] is True: if config.searchPeer(public_key)[0] is True:
@@ -816,14 +812,6 @@ def API_addPeers(configName):
# Check if provided pubkey match provided private key # Check if provided pubkey match provided private key
if public_key != genPub: if public_key != genPub:
return ResponseObject(False, "Provided Public Key does not match provided Private Key") return ResponseObject(False, "Provided Public Key does not match provided Private Key")
# if len(public_key) == 0 and len(private_key) == 0:
# private_key = GenerateWireguardPrivateKey()[1]
# public_key = GenerateWireguardPublicKey(private_key)[1]
# elif len(public_key) == 0 and len(private_key) > 0:
# public_key = GenerateWireguardPublicKey(private_key)[1]
if len(allowed_ips) == 0: if len(allowed_ips) == 0:
if ipStatus: if ipStatus:
for subnet in availableIps.keys(): for subnet in availableIps.keys():
@@ -846,7 +834,7 @@ def API_addPeers(configName):
if not found: if not found:
return ResponseObject(False, f"This IP is not available: {i}") return ResponseObject(False, f"This IP is not available: {i}")
status, result = config.addPeers([ status, addedPeers, message = config.addPeers([
{ {
"name": name, "name": name,
"id": public_key, "id": public_key,
@@ -860,14 +848,11 @@ def API_addPeers(configName):
"advanced_security": "off" "advanced_security": "off"
}] }]
) )
# DashboardWebHooks.RunWebHook('peer_created', { return ResponseObject(status=status, message=message, data=addedPeers)
# "configuration": config.Name,
# "peers": [{"id": public_key}]
# })
return ResponseObject(status=status, message=result['message'], data=result['peers'])
except Exception as e: except Exception as e:
app.logger.error("Add peers failed", data, exc_info=e) app.logger.error("Add peers failed", e)
return ResponseObject(False, "Add peers failed. Please see data for specific issue") return ResponseObject(False,
f"Add peers failed. Reason: {message}")
return ResponseObject(False, "Configuration does not exist") return ResponseObject(False, "Configuration does not exist")

View File

@@ -248,7 +248,7 @@ class AmneziaWireguardConfiguration(WireguardConfiguration):
for i in existingPeers: for i in existingPeers:
self.Peers.append(AmneziaWGPeer(i, self)) self.Peers.append(AmneziaWGPeer(i, self))
def addPeers(self, peers: list) -> tuple[bool, dict]: def addPeers(self, peers: list) -> tuple[bool, list, str]:
result = { result = {
"message": None, "message": None,
"peers": [] "peers": []
@@ -305,10 +305,10 @@ class AmneziaWireguardConfiguration(WireguardConfiguration):
"configuration": self.Name, "configuration": self.Name,
"peers": list(map(lambda k : k['id'], peers)) "peers": list(map(lambda k : k['id'], peers))
}) })
return True, result
except Exception as e: except Exception as e:
result['message'] = str(e) current_app.logger.error("Add peers error", e)
return False, result return False, [], str(e)
return True, result['peers'], ""
def getRestrictedPeers(self): def getRestrictedPeers(self):
self.RestrictedPeers = [] self.RestrictedPeers = []

View File

@@ -513,10 +513,7 @@ class WireguardConfiguration:
}) })
) )
def addPeers(self, peers: list) -> tuple[bool, list, str]:
def addPeers(self, peers: list) -> tuple[bool, dict]:
result = { result = {
"message": None, "message": None,
"peers": [] "peers": []
@@ -571,10 +568,10 @@ class WireguardConfiguration:
"configuration": self.Name, "configuration": self.Name,
"peers": list(map(lambda k : k['id'], peers)) "peers": list(map(lambda k : k['id'], peers))
}) })
return True, result
except Exception as e: except Exception as e:
result['message'] = str(e) current_app.logger.error("Add peers error", e)
return False, result return False, [], str(e)
return True, result['peers'], ""
def searchPeer(self, publicKey): def searchPeer(self, publicKey):
for i in self.Peers: for i in self.Peers: