Why does Weather Master require so many system hooks and observers? #400
Labels
Comments
|
@moonkidofhell Hey! Thanks for the feedback! I checked the codebase and didn’t find any
Yeah, it’s mostly for communication between Java and JS, so it uses a ton of interfaces, mainly for theming.
I'll look into it further and see what I can improve |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
I recently analyzed the behavior of the Weather Master app and noticed it's registering for a large number of Android system broadcasts and content observers that seem unrelated to standard weather functionality.
Observed Broadcast/Event Hooks
android.hardware.usb.action.USB_DEVICE_ATTACHEDandroid.hardware.usb.action.USB_DEVICE_DETACHEDandroid.intent.action.HEADSET_PLUGandroid.intent.action.LOCALE_CHANGEDandroid.intent.action.PROXY_CHANGEandroid.intent.action.TIMEZONE_CHANGEDandroid.security.action.KEYCHAIN_CHANGEDandroid.security.action.KEY_ACCESS_CHANGEDandroid.security.action.TRUST_STORE_CHANGEDcom.example.weathermaster.GPU_PROFILER_STARTcom.example.weathermaster.GPU_PROFILER_STOPInvoked Methods
android.media.AudioManager.getOutputLatencyandroid.os.Trace.asyncTraceBeginandroid.os.Trace.asyncTraceEndandroid.os.Trace.isTagEnabledandroid.os.Trace.traceCounterandroid.security.net.config.RootTrustManager.checkServerTrustedandroid.security.net.config.RootTrustManager.isSameTrustConfigurationandroid.view.textclassifier.logging.SmartSelectionEventTracker.logEventcom.example.weathermaster.MainActivity$1.onHideCustomViewcom.example.weathermaster.MainActivity$1.onShowCustomViewContent Observers Registered
content://settings/global/animator_duration_scalecontent://settings/secure/accessibility_captioning_background_colorcontent://settings/secure/accessibility_captioning_edge_colorcontent://settings/secure/accessibility_captioning_edge_typecontent://settings/secure/accessibility_captioning_enabledcontent://settings/secure/accessibility_captioning_font_scalecontent://settings/secure/accessibility_captioning_foreground_colorcontent://settings/secure/accessibility_captioning_localecontent://settings/secure/accessibility_captioning_presetcontent://settings/secure/accessibility_captioning_typefacecontent://settings/secure/accessibility_captioning_window_colorThese seem excessive for a weather app, especially considering the sensitivity of some hooks (USB attachment, trust store changes, keychain events, etc.).
Questions:
The text was updated successfully, but these errors were encountered: