Validation succeed as long as status code in response is 200

[cliu123]

Description

Data source validation should pass as long as response code is 200 even if response body is empty, for example:

{
  body: '',
  statusCode: 200,
  headers: {
    'content-type': 'text/plain; charset=UTF-8',
    'content-length': '0',
    'x-envoy-upstream-service-time': '44',
    date: 'Wed, 10 Apr 2024 06:10:28 GMT',
    server: 'aoss-amazon-m',
    'x-request-id': '284bf59f-4601-9ce9-ba7b-2c2282e34c17'
  },
  meta: {
    context: null,
    request: { params: [Object], options: {}, id: 1 },
    name: 'opensearch-js',
    connection: {
      url: 'https://vywd1o523wrumu9h3q05.gamma-us-east-1.aoss.amazonaws.com/',
      id: 'https://vywd1o523wrumu9h3q05.gamma-us-east-1.aoss.amazonaws.com/',
      headers: {},
      deadCount: 0,
      resurrectTimeout: 0,
      _openRequests: 0,
      status: 'alive',
      roles: [Object]
    },
    attempts: 0,
    aborted: false
  }
}

Issues Resolved

#6396

Screenshot

Testing the changes

Check List

• All tests pass
  • yarn test:jest
  • yarn test:jest_integration
• New functionality includes testing.
• New functionality has been documented.
• Update CHANGELOG.md
• Commits are signed per the DCO using --signoff

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 45.05%. Comparing base (85df662) to head (e53b3dc).
Report is 12 commits behind head on main.

❗ Current head e53b3dc differs from pull request most recent head d906a28. Consider uploading reports for the commit d906a28 to get more accurate results

Additional details and impacted files

@@             Coverage Diff             @@
##             main    #6399       +/-   ##
===========================================
- Coverage   55.58%   45.05%   -10.54%     
===========================================
  Files        1199     1650      +451     
  Lines       24259    33471     +9212     
  Branches     4087     6359     +2272     
===========================================
+ Hits        13485    15079     +1594     
- Misses      10133    17225     +7092     
- Partials      641     1167      +526     

Flag	Coverage Δ
Linux_3	45.05% <100.00%> (?)
Windows_2	?

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[cliu123]

@bbarani @gaiksaya @kavilla Could we include this fix in 2.13.1?
Cc: @seraphjiang

[cliu123]

This fix will be backported to 2.13 branch for next patch release of 2.13.x.
Discussion: #6396 (comment)

[seraphjiang]

#6396

[seraphjiang]

@xinruiba @bandinib-amzn @Flyingliuhub @ZilongX @BionIT @zhongnansu

please help to take a look when you got time

[xinruiba]

LGTM, thanks for this change