fix: abort connection only when abortConnectionOnPingFailure is true

[sumitvekariya]

Title

fix: abort connection only when abortConnectionOnPingFailure is true

Description

#2678
This implementation does the following:

1. It checks if abortConnectionOnPingFailure is not explicitly set to false.
2. If it's not false (by default true), it aborts the connection.
3. If it's false, it logs a warning but doesn't abort the connection.

Change checklist

• I have performed a self-review of my own code
• I have made corresponding changes to the documentation if necessary (this includes comments as well)
• I have added tests that prove my fix is effective or that my feature works

[achingbrain]

Thank you for opening this. Please can you add a test to prevent future regressions?

[sumitvekariya]

Thank you for opening this. Please can you add a test to prevent future regressions?

test cases added

[maschad]

Thanks for taking this on @sumitvekariya , it seems like you have some build errors

Please ensure you read through the README.md particularly the development section before opening the PR for review so that the PR builds and tests pass before review.

[sumitvekariya]

Thanks for taking this on @sumitvekariya , it seems like you have some build errors

Please ensure you read through the README.md particularly the development section before opening the PR for review so that the PR builds and tests pass before review.

Hey, apologies, I missed the build errors, I have fixed the build errors, and tested locally, build is successful and all the test cases are passed as well