leoek / fetch-to-curl

Convert javascript fetch requests to curl
MIT License
67 stars 13 forks source link

Fix issue with empty headers #37

Closed bkniffler closed 2 years ago

bkniffler commented 2 years ago

If for some reason a header value is undefined/null, we get an error (can't call function replace of undefined). We could filter out empty headers also, but if the header is there and empty, maybe we'd still want it output?

codecov[bot] commented 2 years ago

Codecov Report

Merging #37 (94fb2ef) into master (3af4e6d) will not change coverage. The diff coverage is 100.00%.

Impacted file tree graph

@@            Coverage Diff            @@
##            master       #37   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files            1         1           
  Lines           43        43           
  Branches        18        18           
=========================================
  Hits            43        43           
Impacted Files Coverage Δ
src/main.js 100.00% <100.00%> (ø)

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 3af4e6d...94fb2ef. Read the comment docs.