Skip to content

[crashlytics] Specify the executor when calling onFailureListener - #6535

Merged
rlazo merged 2 commits into
mainfrom
rl.crashlytics.failure.listener
Nov 25, 2024
Merged

rlazo merged 2 commits into
mainfrom
rl.crashlytics.failure.listener

Conversation

@rlazo

@rlazo rlazo commented Nov 25, 2024

Copy link
Copy Markdown
Contributor

To prevent unwanted executions in the main thread, callbacks should specify the executor explicitly.

To prevent unwanted executions in the main thread, callbacks should
specify the executor explicitly.
@rlazo
rlazo requested a review from mrober November 25, 2024 20:33
@github-actions

github-actions Bot commented Nov 25, 2024 •

Copy link
Copy Markdown
Contributor

Release note changes

The following release notes were modified. Please ensure they look correct.

Release Notes
firebase-crashlytics
### {{crashlytics}} version 19.2.2 {: #crashlytics_v19-2-2}

* {{fixed}} Execute failure listener outside the main thread  GitHub [#6535](//github.com/firebase/firebase-android-sdk/issues/6535){: .external}

#### {{crashlytics}} Kotlin extensions version 19.2.2 {: #crashlytics-ktx_v19-2-2}

The Kotlin extensions library transitively includes the updated
`firebase-crashlytics` library. The Kotlin extensions library has no additional
updates.

@github-actions

Copy link
Copy Markdown
Contributor

Vertex AI Mock Responses Check ⚠️

A newer major version of the mock responses for Vertex AI unit tests is available. update_responses.sh should be updated to clone the latest version of the responses: v5.2

@github-actions

github-actions Bot commented Nov 25, 2024 •

Copy link
Copy Markdown
Contributor

Test Results

 8 files   -  1 028   8 suites   - 1 028   36s ⏱️ - 36m 8s
22 tests  -  5 851  22 ✅  -  5 829  0 💤  - 22  0 ❌ ±0 
44 runs   - 11 787  44 ✅  - 11 743  0 💤  - 44  0 ❌ ±0 

Results for commit 8045f22. ± Comparison against base commit 1b24b82.

This pull request removes 5851 tests.
com.google.android.datatransport.cct.CctBackendFactoryTest ‑ create_returnCCTBackend_WhenBackendNameIsCCT
com.google.android.datatransport.cct.CctDestinationTest ‑ cctDestination_shouldOnlySupportProtoAndJson
com.google.android.datatransport.cct.CctDestinationTest ‑ cctDestination_shouldSupportProtoAndJson
com.google.android.datatransport.cct.CctTransportBackendTest ‑ decorate_whenOffline_shouldProperlyPopulateNetworkInfo
com.google.android.datatransport.cct.CctTransportBackendTest ‑ decorate_whenOnline_shouldProperlyPopulateNetworkInfo
com.google.android.datatransport.cct.CctTransportBackendTest ‑ schedule_shouldAddCookieOnPseudonymousIds
com.google.android.datatransport.cct.CctTransportBackendTest ‑ schedule_shouldDropCookieOnMixedPseudonymousIds
com.google.android.datatransport.cct.CctTransportBackendTest ‑ send_CompressedResponseIsUncompressed
com.google.android.datatransport.cct.CctTransportBackendTest ‑ send_whenBackendRedirectsMoreThan5Times_shouldOnlyRedirect4Times
com.google.android.datatransport.cct.CctTransportBackendTest ‑ send_whenBackendRedirects_shouldCorrectlyFollowTheRedirectViaPost
…

♻️ This comment has been updated with latest results.

@google-oss-bot

google-oss-bot commented Nov 25, 2024 •

Copy link
Copy Markdown
Collaborator

@google-oss-bot

Copy link
Copy Markdown
Collaborator

Size Report 1

Affected Products

  • firebase-crashlytics

    TypeBase (1b24b82)Merge (5f60855)Diff
    aar411 kB411 kB+82 B (+0.0%)
    apk (aggressive)698 kB699 kB+96 B (+0.0%)
    apk (release)5.85 MB5.85 MB+140 B (+0.0%)

Test Logs

  1. https://storage.googleapis.com/firebase-sdk-metric-reports/aQVrB7utnQ.html

@rlazo
rlazo merged commit c00de5a into main Nov 25, 2024
@rlazo
rlazo deleted the rl.crashlytics.failure.listener branch November 25, 2024 22:23
@firebase firebase locked and limited conversation to collaborators Dec 26, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants