Skip to content
Snippets Groups Projects
Commit 7d8d32c2 authored by Edward Hicks's avatar Edward Hicks :8ball:
Browse files

remove the try/except from around the node list check - we moved it to

the sanity check function
parent 5b957694
No related branches found
No related tags found
No related merge requests found
Pipeline #92014 passed
......@@ -431,13 +431,9 @@ while True:
# TODO: Add crypto signing support to ensure messages come from legit source
# Only proceed if the message is meant for either everyone or us explicitly
try:
if '*' not in payload['nodes'] and Session['nodename'] not in payload['nodes']:
printlog("Message not meant for us - ignoring it", loglevels.DEBUG)
continue
except:
# 'nodes' isn't a list so ignore the command
raise Exception("Invalid 'nodes' section in message payload")
if '*' not in payload['nodes'] and Session['nodename'] not in payload['nodes']:
printLog("Message not meant for us - ignoring it", loglevels.DEBUG)
continue
# We're go. Get the actual command to execute
try:
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment