[CCI] Connection request method callback

[timursaurus]

Description

The request method of Connection invokes a callback, but required arguments of err are not provided.

Check List

• New functionality includes testing.
  • All tests pass
• Linter check was successfull - yarn run lint doesn't show any errors
• Commits are signed per the DCO using --signoff
• Changelog was updated.

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

[nhtruong]

The error message usually provides additional information on why it was thrown. In this case, it just repeats the name of the error. Does the added error message provide any utility?

[timursaurus]

RequestAbortedError invokes its superclass with message arg:

class RequestAbortedError extends OpenSearchClientError {
  constructor(message, meta) {
    super(message);
    ...
    this.message = message || 'Request aborted';
  }
}

but it invokes it before the default value of message "Request aborted" is set. It's not a big deal, but it raises eslint and typescript errors in the IDE, potentially stopping linting in the future

We also could assign the default value in the constructor

constructor(message = "Request aborted", meta)

Apart from type-safety and better developer experience, it brings no other utility