1
0
mirror of https://github.com/EDCD/EDMarketConnector.git synced 2025-04-18 18:07:37 +03:00

EDDN: EDDNSender.send_message() doesn't care about do/don't send options

By this point other code will have made that decision.
This commit is contained in:
Athanasius 2022-10-05 15:46:12 +01:00 committed by Athanasius
parent 0d35f8874a
commit 7d0ae88757
No known key found for this signature in database
GPG Key ID: 772697E181BB2767

View File

@ -342,6 +342,10 @@ class EDDNSender:
"""
Transmit a fully-formed EDDN message to the Gateway.
If this is called then the attempt *will* be made. This is not where
options to not send to EDDN, or to delay the sending until docked,
are checked.
Should catch and handle all failure conditions. A `True` return might
mean that the message was successfully sent, *or* that this message
should not be retried after a failure, i.e. too large.
@ -349,9 +353,6 @@ class EDDNSender:
:param msg: Fully formed, string, message.
:return: `True` for "now remove this message from the queue"
"""
# TODO: Check if user options require us to send at this time.
should_return, new_data = killswitch.check_killswitch('plugins.eddn.send', json.loads(msg))
if should_return:
logger.warning('eddn.send has been disabled via killswitch. Returning.')