postman-runtime icon indicating copy to clipboard operation
postman-runtime copied to clipboard

Fix NTLM auth state when `iterationCount` is more than 2

Open coditva opened this issue 5 years ago • 1 comments

This fix is kind of a hack because the architecture is not ideal. Verified that tests fail before the fix.

coditva avatar Dec 19 '19 07:12 coditva

Codecov Report

Merging #945 into develop will increase coverage by 0.03%. The diff coverage is 75%.

Impacted file tree graph

@@            Coverage Diff             @@
##           develop    #945      +/-   ##
==========================================
+ Coverage    90.77%   90.8%   +0.03%     
==========================================
  Files           42      42              
  Lines         2373    2381       +8     
  Branches       668     670       +2     
==========================================
+ Hits          2154    2162       +8     
  Misses         219     219
Flag Coverage Δ
#integration 80% <75%> (+0.06%) :arrow_up:
#legacy 58.29% <18.75%> (-0.12%) :arrow_down:
#unit 47.24% <0%> (-0.16%) :arrow_down:
Impacted Files Coverage Δ
lib/runner/extensions/item.command.js 92.85% <100%> (+0.17%) :arrow_up:
lib/authorizer/ntlm.js 89.87% <69.23%> (+0.83%) :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 ef7da34...87179ca. Read the comment docs.

codecov[bot] avatar Dec 19 '19 07:12 codecov[bot]