| FazBrowse GitHub Viewer | Trending | | Home |
| Tools: [Download Repo ZIP] [Original HTTPS Page] |
Sorry, something went wrong.
Detect explicit io.sentry.metrics.enabled manifest values and emit tailored migration warnings without applying the obsolete option. Co-Authored-By: Claude <noreply@anthropic.com>
|
Sorry, something went wrong.
📲 Install BuildsAndroid
|
Sorry, something went wrong.
| @Test | ||
| fun `legacy metrics metadata does not disable capture`() { | ||
| val bundle = bundleOf(ManifestMetadataReader.ENABLE_METRICS to false) | ||
| fun `applyMetadata does not warn when legacy metrics enabled metadata is absent`() { |
There was a problem hiding this comment.
Identical to the same test for logs. Does it make sense to keep both? Or should we merge them together into one test?
Sorry, something went wrong.
| Back | FazBrowse Home | New Git URL |
PR Stack (Logs and Metrics Enable Flags)
📜 Description
Detects explicit legacy io.sentry.metrics.enabled Android manifest metadata and emits tailored migration warnings for both true and false values.
The obsolete metadata value is not applied. Manual Sentry.metrics() capture remains active regardless of the configured value, and absent metadata emits no warning.
💡 Motivation and Context
The aggregate Metrics enable option was removed in the previous PR. Existing Android applications may still configure its manifest key, so a focused warning helps users remove stale configuration without restoring the old behavior.
💚 How did you test it?
📝 Checklist
🔮 Next steps
Add a migration warning for legacy external metrics.enabled configuration.
#skip-changelog