Added timeout from env variable to IB client requests

[marcodambros]

Pull Request

This very small PR include two changes:

1. It fixes a bug in IB data client logger, which always printed None instead of the instrument name
2. For some requests to IB, instead of using a default timeout of 60, the timeout can be set with an environment variable, which if does not exist, will default to 60.

Type of change

Delete options that are not relevant.

• Bug fix (non-breaking change which fixes an issue)
• New feature (non-breaking change which adds functionality)

How has this change been tested?

The code was tested locally by building nautilus and using it as a dependency in a project.

[CLAassistant]

All committers have signed the CLA.

[cjdsellers]

Hi @marcodambros

Thanks for the contribution!

(1) Seems like a good change.
(2) We might want to make this more first-class and incorporate the timeout interval into the configuration, maybe @rsmb7z has some thoughts?

[rsmb7z]

Hi @cjdsellers @marcodambros
I think adding it to InteractiveBrokersDataClientConfig would be more consistent and easier to find configurable params.

[cjdsellers]

Hi @marcodambros

If you can remove these lines sourcing the timeout from an env var, we can then merge this and I'll add a config option for the timeout.

[cjdsellers]

Hi @marcodambros

I've gone ahead and fixed the subscription logging per this PR (and credited you in the release notes).

For the timeout, that's actually exposed as a timeout parameter already on HistoricInteractiveBrokersClient.request_bars, so no need for additional config as far as I can see?

[cjdsellers]

Fixed with commit 52c3ba9.