davidmarkclements / fast-safe-stringify

Safely and quickly serialize JavaScript objects
MIT License
348 stars 27 forks source link

New Default Depth Limit Breaks Things #57

Closed rsquires closed 3 years ago

rsquires commented 3 years ago

Just installed some dependencies that include a lenient semver string for this package. The depth limit made my app break in strange ways, wasn't really expecting [...] to start showing up in my REST calls. The depth limit is a neat feature, but probably shouldn't have a default value.

mcollina commented 3 years ago

You are likely right, sorry for the mishap! We will remove the default value.