expressjs / multer / 33261939160

100%

master: 100%

push

github

fix: do not consume maxCount for files skipped by fileFilter (#1426)

wrappedFileFilter decremented the field's remaining-count before the user
fileFilter ran, so a file skipped via cb(null, false) still consumed a
slot. With e.g. .array('docs', 1), a skipped file made the next accepted
file on the same field fail with LIMIT_UNEXPECTED_FILE, even though zero
files had been stored. Decrement only after the filter accepts the file.

Co-authored-by: Ulises Gascon <ulisesgascongonzalez@gmail.com>

184 of 193 branches covered (95.34%)

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

361 of 362 relevant lines covered (99.72%)

6996.81 hits per line