quickfixj
quickfixj copied to clipboard
Add affected `socketAddress` to `SessionStateListener#onConnectException`
trafficstars
Please, consider adding socketAddress to exception passed as argument for quickfix.SessionStateListener#onConnectException
This information is available in quickfix.mina.initiator.IoSessionInitiator.ConnectTask#handleConnectException method and it can be very useful for troubleshooting outside the library
sounds like a sensible enhancement. Are you able to submit a PR for this?
@chrjohn Sure! Opened a pull request, take a look. Maybe I need to implement some tests for new code?
Thanks for the PR, let's continue the discussion there.