All new code requires tests to ensure against regressions.
However, if your PR contains zero code changes, feel free to select the checkmark below to indicate so.
[X] Have you ran tests against this code?
[ ] This PR contains zero code changes.
Description of the Change
While this likely isn't the best solution, it's the best I could determine on a short notice to remove this error from production.
The error stems from the fact that it seems superagent in it's error returns uses many references to other objects in it's error, which seems to create some situations where an object will have an infinite depth if following all the references that reference parent objects and such.
This quick and dirty method of avoiding reaching any call stack limits just limits the maximum depth we sanitize to 15 levels. While not perfect should work for our use cases.
Requirements
Filling out the template is required.
All new code requires tests to ensure against regressions.
[X] Have you ran tests against this code?
[ ] This PR contains zero code changes.
Description of the Change
While this likely isn't the best solution, it's the best I could determine on a short notice to remove this error from production.
The error stems from the fact that it seems
superagentin it's error returns uses many references to other objects in it's error, which seems to create some situations where an object will have an infinite depth if following all the references that reference parent objects and such.This quick and dirty method of avoiding reaching any call stack limits just limits the maximum depth we sanitize to 15 levels. While not perfect should work for our use cases.
Fixes #252