Sentry-react-native: recursion limit hit

[vtoupet]

OS:

• [ ] Windows
• [x] MacOS
• [ ] Linux

_Platform:_

• [ ] iOS
• [x] Android

Output of node -v && npm -v && npm ls --prod --depth=0

v8.7.0
5.5.1
[email protected] /Users/vincent/Documents/workspace/smartmixin/mobile
├── [email protected]
├── [email protected]
├── [email protected]
├── [email protected]
├── [email protected]
├── [email protected]
├── [email protected]
├── [email protected]
├── [email protected]
├── [email protected]
├── UNMET PEER DEPENDENCY [email protected]
├── [email protected]
├── [email protected]
├── [email protected]
├── [email protected]
├── [email protected]
├── [email protected]
├── [email protected]
├── [email protected]
├── [email protected]
├── [email protected]
├── [email protected]
├── [email protected]
├── [email protected]
├── [email protected] invalid
├── [email protected]
├── [email protected]
├── [email protected]
├── [email protected]
├── UNMET PEER DEPENDENCY [email protected]
├── [email protected]
├── [email protected]
├── [email protected]
├── [email protected]
├── [email protected]
├── [email protected]
├── [email protected]
├── [email protected]
├── [email protected]
└── [email protected]

---

my sentry additional data is truncated like below (it did not happen in the past with same recursive level)

{
account: {
last_updated: 1509095522, 
thermostats: [
<recursion limit hit>, 
<recursion limit hit>
]
}, 
chartParams: {
period: day
}, 
connected: True, 
connectionDate: 1504688318, 
shouldSelectorOpen: False
}

Is there a way to remove this limit or set it to a higher level?

[HazAT]

Can you link to one of these event please?

[vtoupet]

@HazAT https://sentry.io/vincent-toupet/smartmixin-mobile/issues/393497081/events/8464743902/

[vtoupet]

@HazAT any news about that issue?

[HazAT]

Hey, sorry it was so quiet around this.
We had a discussion over here https://github.com/getsentry/raven-js/issues/623
not sure where the <recursion limit hit> is coming from though.
So short answer there will not be a quick fix for this so I will close it for now.
I will post an answer here as soon as we support it.

[ywongweb]

Encountered the same <recursion limit hit> message when sending in additional data with 3+ nested levels.

RN: 0.51.0
react-native-sentry: 0.33.0

[humphriest]

@HazAT Is there any progression on this issue? Is there a workaround with raven or something else?

[thevolcanomanishere]

@HazAT SentryJsonGenerator.json has a max nest level of 3 here. But this doesnt seem to be applied all the time. If we are not hitting the max payload limit, why is data being stripped from the object shown in sentry?

https://github.com/getsentry/sentry-java/blob/027544f8a4063740294bde23073ea409d38ed5e1/sentry/src/main/java/io/sentry/marshaller/json/SentryJsonGenerator.java#L27-L31

[cpetzel]

any update on this?

[humphriest]

Has a solution been found yet for this?

[wikka]

I'm having the same problem and it is affecting the debug of critical issues in an App we are developing.
Does anyone have a recommendation of how to deal with it while it is not fixed?

[askielboe]

Same issue when adding redux state to breadcrumbs. 3rd level values are all recursion limit hit.

Only seen this on Android so far.

[abdourahimjallow]

Any new

[laurentrivard]

bump... ? hitting the same limit

[faizalshap]

Same here...

[iagormoraes]

same here, is showing on a catch error validation from JOI with sub level object.

[Danman7]

Same here...

[HazAT]

I am closing all old issues, please if this is still a problem feel free to revive it.
Also, consider moving to our new SDK @sentry/react-native if it still happens there please open a new issue.