Fix to use 'abort' event instead of 'aborted' property

[briannielson]

Fixes #1455

[briannielson]

Closing due to failing test.

[briannielson]

Tests fail, but they need to be reworked to use a ClientRequest object as specified in web-incoming.js:

  /**
   * Does the actual proxying. If `forward` is enabled fires up
   * a ForwardStream, same happens for ProxyStream. The request
   * just dies otherwise.
   *
   * @param {ClientRequest} Req Request object
   * @param {IncomingMessage} Res Response object
   * @param {Object} Options Config object passed to the proxy
   *
   * @api private
   */

[jayeshmend]

@briannielson any idea when this will be fixed as its creating issues at random interval.

[briannielson]

@briannielson any idea when this will be fixed as its creating issues at random interval.

Sorry, I haven't been watching this PR or the related issue. We ended up patching the library in our codebase with this change. This caused tests to fail in this repo, but I didn't have time to spend on fixing the tests. This did appear to fix the issue in our production environment.