Remove unnecessary if/else clause

function only returns True or raises an Exception for an error
This commit is contained in:
freamon 2024-08-15 11:16:57 +00:00
parent 589ee8413a
commit f247c280e9

View file

@ -432,14 +432,12 @@ def shared_inbox():
actor = find_actor_or_create(request_json['actor']) if 'actor' in request_json else None
if actor is not None:
try:
if HttpSignature.verify_request(request, actor.public_key, skip_date=True):
if current_app.debug:
process_inbox_request(request_json, activity_log.id, ip_address())
else:
process_inbox_request.delay(request_json, activity_log.id, ip_address())
return ''
HttpSignature.verify_request(request, actor.public_key, skip_date=True)
if current_app.debug:
process_inbox_request(request_json, activity_log.id, ip_address())
else:
activity_log.exception_message = 'Could not verify signature'
process_inbox_request.delay(request_json, activity_log.id, ip_address())
return ''
except VerificationError as e:
activity_log.exception_message = 'Could not verify signature: ' + str(e)
else: