perf(reqresp): improve the reqresp request/response methods

[nazarhussain]

Motivation

Avoid conditional parameters inside most frequently used methods.

Description

• For every message on the network sendRequest is used and that includes 4 conditions with ?? on timeout values. These can be avoided.
• The handleRequest is now renamed as sendResponse to make it aligned with counter part function.
• For every response to network message handleRequest was extra conditions for timeout values that can be avoided.

Steps to test or reproduce

• Run all tests

[codecov]

Codecov Report

Attention: Patch coverage is 74.44444% with 23 lines in your changes are missing coverage. Please review.

Project coverage is 61.87%. Comparing base (a6b10df) to head (8fa4fa7).

Additional details and impacted files

@@             Coverage Diff              @@
##           unstable    #6801      +/-   ##
============================================
- Coverage     61.87%   61.87%   -0.01%     
============================================
  Files           562      561       -1     
  Lines         59331    59335       +4     
  Branches       1915     1918       +3     
============================================
+ Hits          36714    36715       +1     
- Misses        22574    22577       +3     
  Partials         43       43              

[wemeetagain]

why did this function signature go from f({a, b, c, d, e}) to f(a, b, {c, modules: {d, e}})

[nazarhussain]

To make it consistent for request and response methods. Now both functions sendResponse and sendRequest have same function signature with similar options.