Log Integrations: Add `SentrySwiftLog` Package
#skip-changelog
:scroll: Description
- Adds the
SentrySwiftLogintegration to theintegrations/logs/sentry-swift-logfolder - Skipping changelog as this is not available to users through this repo
:bulb: Motivation and Context
Closes #5372 Relates to #6902
:green_heart: How did you test it?
- Runs pure swift tests against the current
Package.swiftversion - This loads pre-build artefacts, so we always test against already released versions.
- Should we rather test against current source?
:pencil: Checklist
You have to check all boxes before merging:
- [x] I added tests to verify the changes.
- [x] No new PII added or SDK only sends newly added PII if
sendDefaultPIIis enabled. - [ ] I updated the docs if needed.
- [ ] I updated the wizard if needed.
- [ ] Review from the native team if needed.
- [x] No breaking change or entry added to the changelog.
- [x] No breaking change for hybrid SDKs or communicated to hybrid SDKs.
Codecov Report
:white_check_mark: All modified and coverable lines are covered by tests.
:white_check_mark: Project coverage is 84.936%. Comparing base (3acb318) to head (18e2765).
:warning: Report is 12 commits behind head on main.
:white_check_mark: All tests successful. No failed tests found.
Additional details and impacted files
@@ Coverage Diff @@
## main #6941 +/- ##
=============================================
- Coverage 85.041% 84.936% -0.105%
=============================================
Files 454 457 +3
Lines 27703 27604 -99
Branches 12150 12141 -9
=============================================
- Hits 23559 23446 -113
- Misses 4100 4118 +18
+ Partials 44 40 -4
see 50 files with indirect coverage changes
Continue to review full report in Codecov by Sentry.
Legend - Click here to learn more
Δ = absolute <relative> (impact),ø = not affected,? = missing dataPowered by Codecov. Last update 3acb318...18e2765. Read the comment docs.
Performance metrics :rocket:
| Plain | With Sentry | Diff | |
|---|---|---|---|
| Startup time | 1210.29 ms | 1246.00 ms | 35.71 ms |
| Size | 24.14 KiB | 1.03 MiB | 1.00 MiB |
Baseline results on branch: main
Startup times
| Revision | Plain | With Sentry | Diff |
|---|---|---|---|
| 7969533b517f4c480b3a2c6decc90d4afb44dac2 | 1216.16 ms | 1248.22 ms | 32.05 ms |
| df3872ad0d4069e829177540936ef3a84363b8ea | 1234.37 ms | 1268.72 ms | 34.35 ms |
| 1c4c700b85dc0e7a2a79a2f452d74a283f1b4441 | 1193.28 ms | 1218.88 ms | 25.61 ms |
| 3067c237a11ea5f4c7698408fd3925a54c6ba67c | 1230.48 ms | 1257.90 ms | 27.42 ms |
| 0ecea77a73dfa48ec94e47b3941a6478cf9cc31e | 1191.47 ms | 1216.12 ms | 24.65 ms |
| f262e101660a2e489a414e4d231e26e8de7a0b05 | 1226.88 ms | 1250.39 ms | 23.51 ms |
| 62799929787f2fedf64bbf922cda9e40ff7232b1 | 1213.60 ms | 1241.38 ms | 27.79 ms |
| 73c97128227b5caa1eb72ab32bb9b7d6d2eac458 | 1238.57 ms | 1260.38 ms | 21.80 ms |
| c8dd5e412091848b01e8c4342a7b02ce8f90cb9f | 1217.67 ms | 1242.90 ms | 25.23 ms |
| 7d236393f3ac50d31f94c4e5a01995a615a6e1f7 | 1237.93 ms | 1243.04 ms | 5.11 ms |
App size
| Revision | Plain | With Sentry | Diff |
|---|---|---|---|
| 7969533b517f4c480b3a2c6decc90d4afb44dac2 | 24.14 KiB | 1.02 MiB | 1016.90 KiB |
| df3872ad0d4069e829177540936ef3a84363b8ea | 23.74 KiB | 1.02 MiB | 1016.77 KiB |
| 1c4c700b85dc0e7a2a79a2f452d74a283f1b4441 | 24.14 KiB | 1.02 MiB | 1017.25 KiB |
| 3067c237a11ea5f4c7698408fd3925a54c6ba67c | 23.75 KiB | 928.15 KiB | 904.40 KiB |
| 0ecea77a73dfa48ec94e47b3941a6478cf9cc31e | 24.15 KiB | 1.01 MiB | 1014.90 KiB |
| f262e101660a2e489a414e4d231e26e8de7a0b05 | 23.75 KiB | 913.62 KiB | 889.87 KiB |
| 62799929787f2fedf64bbf922cda9e40ff7232b1 | 23.75 KiB | 891.03 KiB | 867.28 KiB |
| 73c97128227b5caa1eb72ab32bb9b7d6d2eac458 | 23.75 KiB | 908.01 KiB | 884.26 KiB |
| c8dd5e412091848b01e8c4342a7b02ce8f90cb9f | 23.75 KiB | 913.48 KiB | 889.72 KiB |
| 7d236393f3ac50d31f94c4e5a01995a615a6e1f7 | 23.75 KiB | 891.01 KiB | 867.26 KiB |
Previous results on branch: feat/add-swift-log-integration
Startup times
| Revision | Plain | With Sentry | Diff |
|---|---|---|---|
| ec6df71c04340a939daeaa477b5848a0df48ff9e | 1210.16 ms | 1240.33 ms | 30.16 ms |
| b78f1c73666d871e5d39a428b55fba74d7131ad6 | 1213.30 ms | 1242.79 ms | 29.49 ms |
| 5c126cfac332c273e3f28d2634162523069ee96c | 1210.42 ms | 1237.67 ms | 27.25 ms |
| 587a9b5e67f38e3c5a942623b034e861f5ba4197 | 1213.80 ms | 1250.94 ms | 37.14 ms |
| e39cfe2be6619f965eaef5991988162864a9a365 | 1196.87 ms | 1216.73 ms | 19.87 ms |
| b06a36606c54dde547956b4c7fcb79639bebd82a | 1212.46 ms | 1242.20 ms | 29.74 ms |
| 213440f4effdb15bea3aef1e2cf8f65715eb3b31 | 1228.80 ms | 1260.73 ms | 31.93 ms |
App size
| Revision | Plain | With Sentry | Diff |
|---|---|---|---|
| ec6df71c04340a939daeaa477b5848a0df48ff9e | 24.14 KiB | 1.02 MiB | 1019.40 KiB |
| b78f1c73666d871e5d39a428b55fba74d7131ad6 | 24.14 KiB | 1.02 MiB | 1019.59 KiB |
| 5c126cfac332c273e3f28d2634162523069ee96c | 24.14 KiB | 1.02 MiB | 1019.58 KiB |
| 587a9b5e67f38e3c5a942623b034e861f5ba4197 | 24.14 KiB | 1.02 MiB | 1016.89 KiB |
| e39cfe2be6619f965eaef5991988162864a9a365 | 24.14 KiB | 1.02 MiB | 1016.90 KiB |
| b06a36606c54dde547956b4c7fcb79639bebd82a | 24.14 KiB | 1.02 MiB | 1016.90 KiB |
| 213440f4effdb15bea3aef1e2cf8f65715eb3b31 | 24.14 KiB | 1.02 MiB | 1019.58 KiB |
🚨 Detected changes in high risk code 🚨
High-risk code can easily blow up and is hard to test. We had severe bugs in the past. Be extra careful when changing these files, and have an extra careful look at these:
- [ ] .github/file-filters.yml