Skip to content

Conversation

@markushi
Copy link
Member

@markushi markushi commented Oct 29, 2025

When no user is set in the scope, the logger now falls back to using the distinct ID (installation ID on Android) as the user ID in log attributes. This ensures logs always have a user identifier when available - and it also matches the DefaultAndroidEventProcessor behavior.

@github-actions
Copy link
Contributor

github-actions bot commented Oct 29, 2025

Performance metrics 🚀

  Plain With Sentry Diff
Startup time 311.09 ms 377.89 ms 66.80 ms
Size 1.58 MiB 2.12 MiB 551.50 KiB

Baseline results on branch: main

Startup times

Revision Plain With Sentry Diff
27d7cf8 309.43 ms 364.27 ms 54.85 ms
3d205d0 352.15 ms 432.53 ms 80.38 ms
ee747ae 554.98 ms 611.50 ms 56.52 ms
ce0a49e 532.00 ms 609.96 ms 77.96 ms
b3d8889 420.46 ms 453.71 ms 33.26 ms
27d7cf8 306.76 ms 366.66 ms 59.90 ms
806307f 357.85 ms 424.64 ms 66.79 ms
c8125f3 397.65 ms 485.14 ms 87.49 ms
d5a29b6 298.62 ms 391.78 ms 93.16 ms
d364ace 384.53 ms 453.51 ms 68.98 ms

App size

Revision Plain With Sentry Diff
27d7cf8 1.58 MiB 2.12 MiB 549.42 KiB
3d205d0 1.58 MiB 2.10 MiB 532.97 KiB
ee747ae 1.58 MiB 2.10 MiB 530.95 KiB
ce0a49e 1.58 MiB 2.10 MiB 532.94 KiB
b3d8889 1.58 MiB 2.10 MiB 535.07 KiB
27d7cf8 1.58 MiB 2.12 MiB 549.42 KiB
806307f 1.58 MiB 2.10 MiB 533.42 KiB
c8125f3 1.58 MiB 2.10 MiB 532.32 KiB
d5a29b6 1.58 MiB 2.12 MiB 549.37 KiB
d364ace 1.58 MiB 2.11 MiB 539.75 KiB

Previous results on branch: markushi/fix/missing-user-id

Startup times

Revision Plain With Sentry Diff
a097a34 341.23 ms 421.35 ms 80.12 ms
e4c8522 318.31 ms 374.80 ms 56.49 ms

App size

Revision Plain With Sentry Diff
a097a34 1.58 MiB 2.12 MiB 551.51 KiB
e4c8522 1.58 MiB 2.12 MiB 551.40 KiB

@markushi
Copy link
Member Author

@sentry review

@markushi markushi merged commit fcec2f2 into main Oct 30, 2025
61 checks passed
@markushi markushi deleted the markushi/fix/missing-user-id branch October 30, 2025 11:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants