gehtsoft / forex-connect

For
http://www.fxcodebase.com/wiki/index.php/Category:ForexConnect
48 stars 48 forks source link

Segfault due to hard coded timeout values #23

Open jaman opened 4 years ago

jaman commented 4 years ago

SessionStatusListener.py, ResponseListener.py and TableManagerListener.py all have a hard-coded wait(30) in them. Under various circumstances, that wait can be exceeded. In such a case, the system can segfault and core-dump.

There are a number of issues here:

What should happen is: Timeout -> 'Tablemanager timeout awaiting [possibly 5he callstack]. Waited ** seconds without response' Now the client should be able to take that information and, increase the allowed timeout via the configurable value, re-evaluate the code their using to see if there is an issue, or reach out to FXCM to investigate issues with slow processing (in the case of things like orders).