(feat-V7) Log Tracing #4827

lucas-zimerman · 2025-05-13T14:22:05Z

📢 Type of change

Bugfix
New feature
Enhancement
Refactoring

Blocked by:

📜 Description

Add the required changes on the Client in order to capture logs. All platforms will be supported with the initial PR (web, iOS and Android)

💡 Motivation and Context

Fixes #4856

💚 How did you test it?

Locally, with native SDKs disabled for testing the JavaScript integration, and with native SDKs enabled to test the Native integration

Pii

Pii seems to be filtered serverside so we don't need to do anything on the SDK.

📝 Checklist

I added tests to verify changes
No new PII added or SDK only sends newly added PII if sendDefaultPII is enabled
I updated the docs if needed.
I updated the wizard if needed.
All tests passing
No breaking changes

🔮 Next steps

Implement tests
Decide what todo with internal logs sent to logger. (Doesn't seems to be sent)
Redirect BeforeLog from native to JS Layer
Add documentation.

github-actions · 2025-05-13T14:40:23Z

Android (legacy) Performance metrics 🚀

	Plain	With Sentry	Diff
Startup time	418.04 ms	405.11 ms	-12.93 ms
Size	17.75 MiB	19.58 MiB	1.83 MiB

Baseline results on branch: v7

Startup times

Revision	Plain	With Sentry	Diff
`3c99746`	399.51 ms	429.09 ms	29.58 ms
`472960b`	418.84 ms	405.38 ms	-13.46 ms
`f870f2d`	444.67 ms	449.62 ms	4.95 ms

App size

Revision	Plain	With Sentry	Diff
`3c99746`	17.75 MiB	19.58 MiB	1.83 MiB
`472960b`	17.75 MiB	19.58 MiB	1.83 MiB
`f870f2d`	17.75 MiB	19.58 MiB	1.83 MiB

Previous results on branch: lz/v7-test-logs

Startup times

Revision	Plain	With Sentry	Diff
`959bf26`	456.37 ms	449.80 ms	-6.57 ms

App size

Revision	Plain	With Sentry	Diff
`959bf26`	17.75 MiB	19.58 MiB	1.83 MiB

github-actions · 2025-05-13T14:59:26Z

iOS (legacy) Performance metrics 🚀

	Plain	With Sentry	Diff
Startup time	1215.81 ms	1217.98 ms	2.17 ms
Size	2.63 MiB	3.79 MiB	1.16 MiB

Baseline results on branch: v7

Startup times

Revision	Plain	With Sentry	Diff
`3c99746`+dirty	1215.12 ms	1222.31 ms	7.18 ms
`472960b`+dirty	1213.96 ms	1222.58 ms	8.62 ms
`f870f2d`+dirty	1227.18 ms	1232.30 ms	5.12 ms

App size

Revision	Plain	With Sentry	Diff
`3c99746`+dirty	2.63 MiB	3.78 MiB	1.15 MiB
`472960b`+dirty	2.63 MiB	3.79 MiB	1.15 MiB
`f870f2d`+dirty	2.63 MiB	3.79 MiB	1.15 MiB

Previous results on branch: lz/v7-test-logs

Startup times

Revision	Plain	With Sentry	Diff
`959bf26`+dirty	1218.46 ms	1237.24 ms	18.78 ms

App size

Revision	Plain	With Sentry	Diff
`959bf26`+dirty	2.63 MiB	3.78 MiB	1.15 MiB

github-actions · 2025-05-13T15:11:47Z

iOS (new) Performance metrics 🚀

	Plain	With Sentry	Diff
Startup time	1224.22 ms	1226.91 ms	2.69 ms
Size	3.19 MiB	4.36 MiB	1.17 MiB

Baseline results on branch: v7

Startup times

Revision	Plain	With Sentry	Diff
`3c99746`+dirty	1227.65 ms	1228.81 ms	1.16 ms
`472960b`+dirty	1243.67 ms	1233.57 ms	-10.11 ms
`f870f2d`+dirty	1230.08 ms	1238.88 ms	8.80 ms

App size

Revision	Plain	With Sentry	Diff
`3c99746`+dirty	3.19 MiB	4.35 MiB	1.16 MiB
`472960b`+dirty	3.19 MiB	4.36 MiB	1.17 MiB
`f870f2d`+dirty	3.19 MiB	4.36 MiB	1.17 MiB

Previous results on branch: lz/v7-test-logs

Startup times

Revision	Plain	With Sentry	Diff
`959bf26`+dirty	1224.94 ms	1235.58 ms	10.64 ms

App size

Revision	Plain	With Sentry	Diff
`959bf26`+dirty	3.19 MiB	4.35 MiB	1.16 MiB

…into lz/v7-test-logs

…-react-native into lz/v7-test-logs

lucas-zimerman · 2025-05-22T14:42:47Z

logs are now correctly captured on Android,iOS and JS.

lucas-zimerman · 2025-05-26T17:39:53Z

build errors will be fixed on #4860

…into lz/v7-test-logs

…-react-native into lz/v7-test-logs

lucas-zimerman · 2025-05-27T10:42:31Z

packages/core/src/js/wrapper.ts

@@ -182,7 +182,7 @@ export const NATIVE: SentryNativeWrapper = {
          typeof itemHeader.content_type === 'string' ? itemHeader.content_type : 'application/octet-stream';
        bytesPayload = itemPayload;
      } else {
-        bytesContentType = 'application/json';
+        bytesContentType = 'application/vnd.sentry.items.log+json';


Should we use the new content type for all requests or only the ones that contain logs? @krystofwoldrich @kahest
This can create a break changes on self -hosted, and I am not sure which version has support for logging.

on Sentry JavaScript for example, they use application/vnd.sentry.items.log+json by default on all requests

github-actions · 2025-05-27T11:30:49Z

Android (new) Performance metrics 🚀

	Plain	With Sentry	Diff
Startup time	404.85 ms	417.24 ms	12.39 ms
Size	7.15 MiB	8.34 MiB	1.18 MiB

Baseline results on branch: v7

Startup times

Revision	Plain	With Sentry	Diff
`472960b`+dirty	394.39 ms	376.18 ms	-18.20 ms
`3c99746`+dirty	400.65 ms	399.59 ms	-1.06 ms
`f870f2d`+dirty	398.49 ms	434.24 ms	35.75 ms

App size

Revision	Plain	With Sentry	Diff
`472960b`+dirty	7.15 MiB	8.34 MiB	1.18 MiB
`3c99746`+dirty	7.15 MiB	8.34 MiB	1.18 MiB
`f870f2d`+dirty	7.15 MiB	8.34 MiB	1.18 MiB

Previous results on branch: lz/v7-test-logs

Startup times

Revision	Plain	With Sentry	Diff
`959bf26`+dirty	380.57 ms	381.71 ms	1.14 ms

App size

Revision	Plain	With Sentry	Diff
`959bf26`+dirty	7.15 MiB	8.34 MiB	1.18 MiB

lucas-zimerman · 2025-05-27T11:47:29Z

CHANGELOG.md

@@ -61,6 +84,10 @@ Version 7 of the SDK is compatible with Sentry self-hosted versions 24.4.2 or hi
 - `@sentry/utils` package, the exports were moved to `@sentry/core`
 - Standalone `Client` interface & deprecate `BaseClient`

+### Self Hosted
+
+- It is recommended to use Sentry Self Hosted version `24.8.0` this and future versions.


I validated that capture of events/exceptions are working on 24.8.0, but we should at least recommend the first version suggested here getsentry/self-hosted#3560

CHANGELOG.md

lucas-zimerman · 2025-05-27T12:07:27Z

CHANGELOG.md

@@ -61,6 +84,10 @@ Version 7 of the SDK is compatible with Sentry self-hosted versions 24.4.2 or hi
 - `@sentry/utils` package, the exports were moved to `@sentry/core`
 - Standalone `Client` interface & deprecate `BaseClient`

+### Self Hosted
+
+- It is recommended to use Sentry Self Hosted version `25.2.0` or new for React Native V7 or newer


this change affects all events/transactions due to the content type of the request being changed.

25.2.0 is a safe value for it getsentry/self-hosted#3560, even if we don't officially support logs, it'll not drop other types of content

samples/react-native/src/App.tsx

lucas-zimerman · 2025-05-28T14:55:29Z

@krystofwoldrich @antonis it's ready for review :D

CHANGELOG.md

antonis

LGTM and worked as expected in my tests 🎸

lucas-zimerman · 2025-06-11T17:51:51Z

As tested here #4911 the failed tests are unrelated to this PR

lucas-zimerman changed the base branch from main to v7 May 13, 2025 14:22

lucas-zimerman added 2 commits May 13, 2025 15:19

add experimental support for log tracing

20cd6e9

project cleanup

b177094

lucas-zimerman added Blocked Feature and removed Blocked labels May 20, 2025

lucas-zimerman added 4 commits May 21, 2025 07:41

Merge branch 'v7' into lz/v7-test-logs

5d432d6

Merge branch 'v7' of https://github.com/getsentry/sentry-react-native …

7c24aa5

…into lz/v7-test-logs

enable android log

14ad53a

Merge branch 'lz/v7-test-logs' of https://github.com/getsentry/sentry…

cc52bf8

…-react-native into lz/v7-test-logs

lucas-zimerman added 2 commits May 22, 2025 15:39

use correct content type

a78e311

Merge branch 'v7' into lz/v7-test-logs

9eb1fb4

lucas-zimerman removed the Blocked label May 26, 2025

lucas-zimerman mentioned this pull request May 26, 2025

chore(deps-v7): update Javascript SDK to v9.22.0 #4860

Merged

10 tasks

lucas-zimerman added 3 commits May 26, 2025 17:05

Merge branch 'v7' of https://github.com/getsentry/sentry-react-native …

307ccd1

…into lz/v7-test-logs

fix lint

d83d2a2

Merge branch 'lz/v7-test-logs' of https://github.com/getsentry/sentry…

0f2f574

…-react-native into lz/v7-test-logs

lucas-zimerman commented May 27, 2025

View reviewed changes

CHANGELOG.md Outdated Show resolved Hide resolved

lucas-zimerman commented May 27, 2025

View reviewed changes

samples/react-native/src/App.tsx Outdated Show resolved Hide resolved

Update samples/react-native/src/App.tsx

c0c3818

lucas-zimerman changed the title ~~(WIP) Log Tracing~~ (feat-V7) Log Tracing May 27, 2025

lucas-zimerman added 9 commits May 27, 2025 11:06

merge v7 / fix yarn lock

310aeb0

fix parenthesis

03c3cfb

avoid var

600f436

add chcangelog

d59fce3

fix tests an add self hosted requirement

ab99989

actually mention 25.2.0

9cb856d

Update CHANGELOG.md

310b00b

Merge branch 'v7' into lz/v7-test-logs

0982fad

Merge branch 'v7' into lz/v7-test-logs

3ffd7ca

lucas-zimerman marked this pull request as ready for review May 28, 2025 10:28

lucas-zimerman requested review from krystofwoldrich and antonis as code owners May 28, 2025 10:28

Merge branch 'v7' into lz/v7-test-logs

c917647

antonis reviewed May 29, 2025

View reviewed changes

CHANGELOG.md Outdated Show resolved Hide resolved

antonis approved these changes May 29, 2025

View reviewed changes

lucas-zimerman added 9 commits June 3, 2025 11:40

Merge branch 'v7' into lz/v7-test-logs

2efdcc9

Merge branch 'v7' into lz/v7-test-logs

1225323

add full snippet

fc591b5

move changelog to unreleased :

65c222d

fix lint

c670e44

Merge branch 'v7' into lz/v7-test-logs

34fb837

Merge branch 'v7' into lz/v7-test-logs

79ca49c

moved changelog to unreleased

70cbd55

moved self hosted warning

026cc10

lucas-zimerman merged commit 8af9fb7 into v7 Jun 11, 2025
80 of 84 checks passed

lucas-zimerman deleted the lz/v7-test-logs branch June 11, 2025 19:06

antonis mentioned this pull request Jun 17, 2025

feat(logs): Add support for logs getsentry/sentry-capacitor#920

Merged

Uh oh!

(feat-V7) Log Tracing #4827

(feat-V7) Log Tracing #4827

Uh oh!

Conversation

lucas-zimerman commented May 13, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

📢 Type of change

Blocked by:

📜 Description

💡 Motivation and Context

💚 How did you test it?

Pii

📝 Checklist

🔮 Next steps

Uh oh!

github-actions bot commented May 13, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Android (legacy) Performance metrics 🚀

Baseline results on branch: v7

Startup times

App size

Previous results on branch: lz/v7-test-logs

Startup times

App size

Uh oh!

github-actions bot commented May 13, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

iOS (legacy) Performance metrics 🚀

Baseline results on branch: v7

Startup times

App size

Previous results on branch: lz/v7-test-logs

Startup times

App size

Uh oh!

github-actions bot commented May 13, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

iOS (new) Performance metrics 🚀

Baseline results on branch: v7

Startup times

App size

Previous results on branch: lz/v7-test-logs

Startup times

App size

Uh oh!

lucas-zimerman commented May 22, 2025

Uh oh!

lucas-zimerman commented May 26, 2025

Uh oh!

lucas-zimerman May 27, 2025

Choose a reason for hiding this comment

Uh oh!

lucas-zimerman May 28, 2025

Choose a reason for hiding this comment

Uh oh!

github-actions bot commented May 27, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Android (new) Performance metrics 🚀

Baseline results on branch: v7

Startup times

App size

Previous results on branch: lz/v7-test-logs

Startup times

App size

Uh oh!

lucas-zimerman May 27, 2025

Choose a reason for hiding this comment

Uh oh!

Uh oh!

lucas-zimerman May 27, 2025

Choose a reason for hiding this comment

Uh oh!

Uh oh!

lucas-zimerman commented May 28, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

Uh oh!

antonis left a comment

Choose a reason for hiding this comment

lucas-zimerman commented May 13, 2025 •

edited

Loading

github-actions bot commented May 13, 2025 •

edited

Loading

github-actions bot commented May 13, 2025 •

edited

Loading

github-actions bot commented May 13, 2025 •

edited

Loading

github-actions bot commented May 27, 2025 •

edited

Loading

lucas-zimerman commented May 28, 2025 •

edited

Loading