commercetools / nodejs

Everything related to the Node.js ecosystem for the commercetools platform.
https://commercetools.github.io/nodejs/
MIT License
75 stars 70 forks source link

Add Request Timeout for `sdk-middleware-auth` #1716

Closed ajimae closed 3 years ago

ajimae commented 3 years ago

Summary

Add timeout functionality to sdk-middleware-auth to timeout process if request exceeds a specified time.

Description

Todo

codecov[bot] commented 3 years ago

Codecov Report

Merging #1716 (0d0b31d) into master (1167644) will increase coverage by 0.00%. The diff coverage is 100.00%.

Impacted file tree graph

@@           Coverage Diff           @@
##           master    #1716   +/-   ##
=======================================
  Coverage   98.66%   98.66%           
=======================================
  Files         128      128           
  Lines        3434     3449   +15     
  Branches      795      801    +6     
=======================================
+ Hits         3388     3403   +15     
  Misses         42       42           
  Partials        4        4           
Impacted Files Coverage Δ
.../sdk-middleware-auth/src/anonymous-session-flow.js 77.77% <ø> (ø)
...sdk-middleware-auth/src/client-credentials-flow.js 77.77% <ø> (ø)
packages/sdk-middleware-auth/src/password-flow.js 77.77% <ø> (ø)
...ages/sdk-middleware-auth/src/refresh-token-flow.js 77.77% <ø> (ø)
packages/sdk-auth/src/tokenProvider.js 100.00% <100.00%> (ø)
packages/sdk-middleware-auth/src/base-auth-flow.js 96.66% <100.00%> (+1.11%) :arrow_up:

Continue to review full report at Codecov.

Legend - Click here to learn more Δ = absolute <relative> (impact), ø = not affected, ? = missing data Powered by Codecov. Last update 1167644...0d0b31d. Read the comment docs.