Skip to content

[QA] Replace Calendar.getInstance with System.currentTimeMillis for breadcrumb ctor #3736

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.

Sign up for GitHub

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

Merged
romtsn merged 8 commits into from
Oct 3, 2024
Merged

[QA] Replace Calendar.getInstance with System.currentTimeMillis for breadcrumb ctor #3736

Changes from 1 commit
Commits
Show all changes
8 commits
Select commit Hold shift + click to select a range
78a7eb7
Replace Calendar getInstance with system currentTimeMillis for breadc…
romtsn Sep 30, 2024
4861467
Revert
romtsn Sep 30, 2024
01ebbf7
Merge branch 'main' into rz/fix/breadcrumb-date-ctor
markushi Oct 2, 2024
c2d497f
Keep Date getTImestamp for compatibility
romtsn Oct 2, 2024
3bdcf92
Changelog
romtsn Oct 2, 2024
663d6aa
Revert
romtsn Oct 2, 2024
e08f61f
Use getTimestamp()
romtsn Oct 2, 2024
e810349
Preserve Date passed in ctor
romtsn Oct 3, 2024
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Changelog
  • Loading branch information
@romtsn
romtsn committed Oct 2, 2024
commit 3bdcf9278545db5e98fad1312bc7b8b86f214c85
1 change: 1 addition & 0 deletions CHANGELOG.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -19,6 +19,7 @@
- For example, (this is already a default behavior) to redact all `TextView`s and their subclasses (`RadioButton`, `EditText`, etc.): `options.experimental.sessionReplay.addRedactViewClass("android.widget.TextView")`
- If you're using code obfuscation, adjust your proguard-rules accordingly, so your custom view class name is not minified
- Fix ensure Application Context is used even when SDK is initialized via Activity Context ([#3669](https://github.com/getsentry/sentry-java/pull/3669))
- Fix potential ANRs due to `Calendar.getInstance` usage in Breadcrumbs constructor ([#3736](https://github.com/getsentry/sentry-java/pull/3736))

*Breaking changes*:

Expand Down