Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

chore(deps): update Native SDK to v0.7.4 #3441

Closed

Conversation

github-actions[bot]
Copy link
Contributor

Bumps sentry-android-ndk/sentry-native from 0.7.0 to 0.7.4.

Auto-generated by a dependency updater.

Changelog

0.7.4

Fixes:

  • Allow crashpad to run under Epic's Anti-Cheat Client by deferring the full crashpad_handler access rights to the client application until a crash occurred. (#980, [crashpad#99](https://github-redirect.dependabot.com/getsentry/crashpad/pull/99))
  • Reserve enough stack space on Windows for our handler to run when the stack is exhausted from stack-overflow. (#982)
  • Only configure a sigaltstack in inproc if no previous configuration exists on Linux and Android. (#982)
  • Store transaction data in the event property extra since the data property is discarded by relay. (#986)

Docs:

  • Add compile-time flag SENTRY_TRANSPORT_COMPRESSION description to the README.md file. (#976)

Internal:

  • Move sentry-android-ndk JNI related parts from sentry-java to sentry-native (#944)
    This will create a pre-built io.sentry:sentry-native-ndk maven artifact, suitable for being consumed by Android apps.

Thank you:

0.7.2

Features:

Fixes:

  • Fix the Linux build when targeting RISC-V. (#972)

Thank you:

0.7.1

Features:

  • Add user feedback capability to the Native SDK. (#966)

Internal:

Docs:

  • Add usage of the breadcrumb data property to the example. #951

@github-actions github-actions bot added the dependencies Pull requests that update a dependency file label May 24, 2024
@bruno-garcia bruno-garcia force-pushed the deps/sentry-android-ndk/sentry-native/0.7.4 branch from a776d66 to dde420f Compare May 24, 2024 03:03
Copy link
Contributor Author

Performance metrics 🚀

  Plain With Sentry Diff
Startup time 337.31 ms 410.90 ms 73.59 ms
Size 1.70 MiB 2.28 MiB 592.87 KiB

Baseline results on branch: main

Startup times

Revision Plain With Sentry Diff
0bd723b 375.20 ms 452.41 ms 77.20 ms
bc4be3b 360.40 ms 435.04 ms 74.64 ms
4e260b3 378.73 ms 454.18 ms 75.45 ms
28c9a83 376.21 ms 446.27 ms 70.06 ms
8838e01 387.41 ms 467.00 ms 79.59 ms
a33b076 390.44 ms 460.29 ms 69.85 ms
619c9b9 409.04 ms 485.88 ms 76.84 ms
937879e 417.64 ms 550.45 ms 132.81 ms
93a76ca 391.54 ms 475.65 ms 84.11 ms
5e04ee8 302.68 ms 343.36 ms 40.68 ms

App size

Revision Plain With Sentry Diff
0bd723b 1.72 MiB 2.29 MiB 578.09 KiB
bc4be3b 1.72 MiB 2.29 MiB 576.53 KiB
4e260b3 1.72 MiB 2.27 MiB 554.95 KiB
28c9a83 1.70 MiB 2.28 MiB 592.00 KiB
8838e01 1.72 MiB 2.29 MiB 578.15 KiB
a33b076 1.70 MiB 2.28 MiB 592.32 KiB
619c9b9 1.70 MiB 2.28 MiB 592.12 KiB
937879e 1.72 MiB 2.27 MiB 558.42 KiB
93a76ca 1.72 MiB 2.29 MiB 576.75 KiB
5e04ee8 1.70 MiB 2.27 MiB 584.64 KiB

Copy link
Member

@markushi markushi left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Let's leave this open for now but not merge it, as 0.7.4 should be part of the 8.0.0 release

@stefanosiano
Copy link
Member

stefanosiano commented Jun 25, 2024

closing in favor of #3476 (in v8)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dependencies Pull requests that update a dependency file
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants