| FazBrowse GitHub Viewer | Trending | | Home |
| Tools: [Download Repo ZIP] [Original HTTPS Page] |
Sorry, something went wrong.
Detect explicit io.sentry.logs.enabled Android manifest metadata and emit tailored migration warnings without changing the integration-local Logs options. Co-Authored-By: Claude <noreply@anthropic.com>
|
Sorry, something went wrong.
📲 Install BuildsAndroid
|
Sorry, something went wrong.
Performance metrics 🚀
Baseline results on branch: feat/remove-logs-enable-flagStartup times
App size
|
Sorry, something went wrong.
| final boolean enableLogs = readBool(metadata, logger, ENABLE_LOGS, false); | ||
| if (enableLogs) { | ||
| logger.log( | ||
| SentryLevel.WARNING, |
There was a problem hiding this comment.
Who reads logs these days? I get the feeling nobody will see it.
Sorry, something went wrong.
There was a problem hiding this comment.
LGTM
Sorry, something went wrong.
| Back | FazBrowse Home | New Git URL |
PR Stack (Logs and Metrics Enable Flags)
📜 Description
Detects explicit io.sentry.logs.enabled Android manifest metadata and emits a tailored migration warning for both true and false values.
The legacy value is read only to select the warning. It does not change Timber or Logcat integration-local opt-ins, and absent metadata emits no warning.
💡 Motivation and Context
The aggregate Logs enable flag was removed in the previous PR. Android applications may still carry the old manifest key, so a targeted startup warning explains the new behavior and points users toward integration-local opt-ins without silently applying obsolete configuration.
💚 How did you test it?
📝 Checklist
🔮 Next steps
Add equivalent migration warnings for legacy external and Spring Boot Logs configuration.
#skip-changelog