newrelic / newrelic-android-agent

SDK to enable instrumentation of Android mobile apps in New Relic
Apache License 2.0
13 stars 12 forks source link

[NR-266264] Connect agent to live harvest v5 response #230

Closed cthomas-newrelic closed 3 months ago

cthomas-newrelic commented 3 months ago

NR-266264 Implement v5/connect request map headers

Moves to /mobile/v5/connect endpoint. Integrates new request_map_headers property on connect response.

NR-251338 Add log/supportability metrics to count AEI

Logs count of visited or skipped AEI records. Adds:

NR-266264 Update v5/connect response fields

The v5 spec adds/removes fields from the expected HarvestConfiguration class. Most changes are benign and fall back to default values, others like application ID was used in agent code.

Refactors harvest components to add a HarvestConfugurable interface, to normalize the exchange of config data. Fixes misnamed config elements. Removes unused properties form SavedState.

codecov[bot] commented 3 months ago

Codecov Report

Attention: Patch coverage is 77.55102% with 33 lines in your changes missing coverage. Please review.

Project coverage is 56.06%. Comparing base (4659f4b) to head (673bedd).

Files Patch % Lines
...ic/agent/android/harvest/HarvestConfiguration.java 68.42% 2 Missing and 10 partials :warning:
...newrelic/agent/android/ApplicationExitMonitor.java 78.84% 7 Missing and 4 partials :warning:
...a/com/newrelic/agent/android/AndroidAgentImpl.java 33.33% 3 Missing and 1 partial :warning:
...lic/agent/android/harvest/HarvestConfigurable.java 0.00% 3 Missing :warning:
...in/java/com/newrelic/agent/android/SavedState.java 70.00% 3 Missing :warning:
Additional details and impacted files ```diff @@ Coverage Diff @@ ## develop #230 +/- ## ============================================= + Coverage 55.89% 56.06% +0.16% - Complexity 3226 3247 +21 ============================================= Files 323 324 +1 Lines 16423 16465 +42 Branches 1791 1795 +4 ============================================= + Hits 9180 9231 +51 + Misses 6270 6251 -19 - Partials 973 983 +10 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.