Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

for reqExecutions execFilter cannot be empty #137

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

charlesverge
Copy link

Description

reqExecutionsAsync will time out if the execFilter is left blank due to the api not responding to the request. At minimum it needs the client id. This change sets a default to the execFilter to prevent the client from hanging. It may not solve all issues mentioned in #82

Testing

Execute connectAsync with raiseSyncError = True and execute reqExecutionsAsync with no filter.

from ib_async import IB
import asyncio


async def main():
    app = IB()

    await app.connectAsync("127.0.0.1", 4001, clientId=5, raiseSyncErrors=True)
    executions = await app.reqExecutionsAsync()
    print("Executions:", executions)
    app.disconnect()


asyncio.run(main())

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant