chore: Re-enable DD crash reporting - WPB-27522 - #5072
Conversation
There was a problem hiding this comment.
Pull request overview
This PR re-enables Datadog crash reporting (without turning Datadog RUM back on) in the iOS app, and adds supporting developer/CI workflows to validate and symbolicate crash reports.
Changes:
- Enable Datadog Crash Reporting in
WireDatadoginitialization. - Add a Developer Tools action to intentionally trigger a crash for validation.
- Enable (and implement) dSYM upload to Datadog in select Fastlane lanes to support symbolication.
Reviewed changes
Copilot reviewed 3 out of 3 changed files in this pull request and generated 2 comments.
| File | Description |
|---|---|
| WireAnalytics/Sources/WireDatadog/WireDatadog.swift | Re-enables Datadog crash reporting by enabling the CrashReporting feature. |
| wire-ios/Wire-iOS/Sources/Developer/DeveloperTools/DeveloperToolsViewModel.swift | Adds a Developer Tools button to trigger a deliberate crash for testing crash reporting. |
| fastlane/Fastfile | Adds a Fastlane switch + helper to upload dSYM zip files to Datadog for symbolication. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| def upload_dsyms_to_datadog(options) | ||
| build = Build.new(options: options) | ||
|
|
||
| upload_symbols_to_datadog( | ||
| api_key: ENV["DATADOG_API_KEY"], | ||
| dsym_paths: [ | ||
| "#{build.artifact_path(with_filename: true)}.app.dSYM.zip" | ||
| ], | ||
| site: "datadoghq.eu" | ||
| ) | ||
| end |
Test Results1 951 tests 1 924 ✅ 2m 32s ⏱️ Results for commit fe59d84. Summary: workflow run #30281087890 |
|
This PR is stale because it has been open 30 days with no activity. Please update it or close it in case is not relevant anymore. |
Issue
This PR re-enables DD crash reporting without RUM. It partially reverts #4407
Checklist
[WPB-XXX].