Fast safe stringify has new default limits

ladjs / superagent

Ajax for Node.js and browsers (JS HTTP client). Maintained for @forwardemail, @ladjs, @spamscanner, @breejs, @cabinjs, and @lassjs.

https://ladjs.github.io/superagent/

MIT License

16.58k stars 1.33k forks source link

Fast safe stringify has new default limits #1641

Closed dpodsiadlo closed 2 years ago

dpodsiadlo commented 2 years ago

Hi,

The fast-safe-stringify guys just have released new version 2.1.0 which has new default limits in the stringify method: https://github.com/davidmarkclements/fast-safe-stringify/compare/v2.0.8...v2.1.0#diff-e727e4bdf3657fd1d798edcd6b099d6e092f8573cba266154583a746bba0f346R22

Since you're not providing any limits in the Request.end method, the newly introduced ones are messing bigger payloads... Any chance to lock fast-safe-stringify to v2.0.8?

dpodsiadlo commented 2 years ago

fast-safe-stringify guys released a fix in version 2.1.1