stanford-oval / thingpedia-api / 67

79%

push

travis-ci

Fix bug in resubmitting POST requests after redirect

HTTP spec says that after a 301, 302 or 307 redirect, POST requests
should be sent with the original body.
Instead, we were sending undefined.

lgtm.com caught this bug by noticing that the variable data we used
was not the one in the outer scope (as we thought) but another
unrelated variable with the same name, that was hoisted earlier.
Replacing "var" with "let" removes the hoisting and fixes the bug.

126 of 210 branches covered (60.0%)

Branch coverage included in aggregate %.

1 of 1 new or added line in 1 file covered. (100.0%)

298 of 418 relevant lines covered (71.29%)

5.62 hits per line