Don't use recursion in TrimmedThrowableData to avoid stack overflow

[mrober]

Don't use recursion in TrimmedThrowableData. This fixes an issue when exceptions with long caused by chains cause a stack overflow.

Pair with PolinaGo

[github-actions]

📝 PRs merging into main branch

Our main branch should always be in a releasable state. If you are working on a larger change, or if you don't want this change to see the light of the day just yet, consider using a feature branch first, and only merge into the main branch when the code complete and ready to be released.

[mrober]

#5102

[google-oss-bot]

Coverage Report ¹

Affected Products

• firebase-crashlytics

  Overall coverage changed from 9.71% (c3b8210) to 9.69% (ed9e578) by -0.02%.

  Filename	Base (c3b8210)	Merge (ed9e578)	Diff
  CrashlyticsReportDataCapture.java	3.46%	3.45%	-0.01%

Test Logs

• Base (c3b8210): https://github.com/firebase/firebase-android-sdk/actions/runs/6114621129
• Merge (ed9e578): https://github.com/firebase/firebase-android-sdk/actions/runs/6125410716

1. https://storage.googleapis.com/firebase-sdk-metric-reports/Isz2mtof8P.html

[github-actions]

Unit Test Results

  6 files  +  6    6 suites  +6   17s ⏱️ +17s
18 tests +18  18 ✔️ +18  0 💤 ±0  0 ❌ ±0 
36 runs  +36  36 ✔️ +36  0 💤 ±0  0 ❌ ±0 

Results for commit 25e9e15. ± Comparison against base commit c3b8210.

[google-oss-bot]

Size Report ¹

Affected Products

• firebase-crashlytics

  Type	Base (c3b8210)	Merge (ed9e578)	Diff
  aar	359 kB	359 kB	+381 B (+0.1%)
  apk (aggressive)	433 kB	433 kB	+84 B (+0.0%)
  apk (release)	2.17 MB	2.17 MB	-220 B (-0.0%)

Test Logs

• Base (c3b8210): https://github.com/firebase/firebase-android-sdk/actions/runs/6114621129
• Merge (ed9e578): https://github.com/firebase/firebase-android-sdk/actions/runs/6125410716

1. https://storage.googleapis.com/firebase-sdk-metric-reports/AS29gyBZNM.html

[aloysiusralph]

@mrober Thanks for addressing this. Any idea when this will be released?