Closed MikeHathaway closed 3 years ago
Merging #205 (a6665f4) into master (79c1f72) will not change coverage. The diff coverage is
n/a
.
@@ Coverage Diff @@
## master #205 +/- ##
=============================
=============================
Continue to review full report at Codecov.
Legend - Click here to learn more
Δ = absolute <relative> (impact)
,ø = not affected
,? = missing data
Powered by Codecov. Last update 79c1f72...d1883d2. Read the comment docs.
Attempting to resolve an issue wherein an order is placed with a successful response, immediately followed by a second response message that the successfully placed order would violate post only requirements. This edge case should be currently handled by checking
messge.get(b'5001')
is not None, since all such INVALID ALO or cancellation rejection messages contain this field.Since the shutdowns are still occuring, even with this check on both
sync_orders
andwait_for_order_processing_response
, this is making me think there is an asynchronicity issue from these reject messages not being queued prior to the unnecessary attempted cancellation.