-
Notifications
You must be signed in to change notification settings - Fork 44
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
Fix NullPointerException issues getting advertising id #1550
Conversation
bc4a1bb
to
3ca41e4
Compare
Codecov ReportAttention:
Additional details and impacted files@@ Coverage Diff @@
## main #1550 +/- ##
==========================================
- Coverage 84.14% 84.08% -0.06%
==========================================
Files 218 218
Lines 7196 7201 +5
Branches 1007 1007
==========================================
Hits 6055 6055
- Misses 750 755 +5
Partials 391 391 ☔ View full report in Codecov by Sentry. |
**This is an automatic release.** ### Bugfixes * Fix NullPointerException issues getting advertising id (#1550) via Toni Rico (@tonidero) ### Dependency Updates * Bump fastlane from 2.218.0 to 2.219.0 (#1549) via dependabot[bot] (@dependabot[bot]) Co-authored-by: revenuecat-ops <ops@revenuecat.com>
### Description We bumped the ad library version in #1550. However, that version of the library adds the `AD_ID` permission, forcing our customers to have it as well, which is not intended. This reverts that bump but keeps the null pointer exception handling.
Description
This was reported in #1548. I wasn't able to reproduce but from looking around, seems that an update to the the latest version of the library should fix it. However, the latest version is pretty old (from 1/2022). Just in case, I added a catch to that block of code with a log, so it doesn't crash.