fix: salesforce: handle ECONNABORTED error #2732

anantjain45823 · 2023-10-17T09:42:35Z

Description of the change

Resolves INT-852
We are getting multiple alerts over many days for ECONNABORTED error which is some network error form salesforce and should be retried from our end.
This PR has a check to convert this to retryable error

Type of change

Bug fix (non-breaking change that fixes an issue)
New feature (non-breaking change that adds functionality)
Breaking change (fix or feature that would cause existing functionality to not work as expected)

Related issues

Fix #1

Checklists

Development

Lint rules pass locally
The code changed/added as part of this pull request has been covered with tests
All tests related to the changed code pass in development

Code review

This pull request has a descriptive title and information useful to a reviewer. There may be a screenshot or screencast attached
"Ready for review" label attached to the PR and reviewers mentioned in a comment
Changes have been reviewed by at least one other engineer
Issue from task tracker has a link to this pull request

codecov · 2023-10-17T09:49:47Z

Codecov Report

Attention: 1 lines in your changes are missing coverage. Please review.

Comparison is base (a54075a) 87.28% compared to head (f6674a7) 87.23%.
Report is 20 commits behind head on develop.

Additional details and impacted files

@@             Coverage Diff             @@
##           develop    #2732      +/-   ##
===========================================
- Coverage    87.28%   87.23%   -0.05%     
===========================================
  Files          643      698      +55     
  Lines        28552    28643      +91     
  Branches      6780     6790      +10     
===========================================
+ Hits         24921    24987      +66     
- Misses        3302     3326      +24     
- Partials       329      330       +1