search

mscdex / busboy

A streaming parser for HTML form data for node.js
MIT License
2.86k stars 213 forks source link

[Bug] try/catch err resolution for older node version #305

Closed elvis-lam closed 2 years ago

elvis-lam commented 2 years ago

https://github.com/mscdex/busboy/blob/9aadb7afbcb8c70c81c93b1018313c1b1835afb0/lib/utils.js#L469

} catch (err) { console.error(err) }

mscdex commented 2 years ago

This isn't a problem if you use a required node version as indicated in the README, which is node v10.16.0+ as of this writing. Additionally if you install with npm (and possibly other package managers) using an older/unsupported node version, it should display a warning during installation.