java.lang.IllegalAccessException: Tried to access visual service WindowManager from a non-visual Context

[gopidholakia]

App crash while app restart and tries to call

countly.onStart(activity)

Error:

Tried to access visual service WindowManager from a non-visual Context:com.company.product.messages.app.MessagesApp@ee05bed WindowManager should be accessed from Activity or other visual Context. Use an Activity or a Context created with Context#createWindowContext(int, Bundle), which are adjusted to the configuration and visual bounds of an area on screen. java.lang.IllegalAccessException: Tried to access visual service WindowManager from a non-visual Context:com.company.product.messages.app.MessagesApp@ee05bed at android.app.ContextImpl.getSystemService(ContextImpl.java:2207) at android.content.ContextWrapper.getSystemService(ContextWrapper.java:934) at ly.count.android.sdk.DeviceInfo$1.getResolution(DeviceInfo.java:127) at ly.count.android.sdk.DeviceInfo.getCommonMetrics(DeviceInfo.java:550) at ly.count.android.sdk.DeviceInfo.getMetrics(DeviceInfo.java:592) at ly.count.android.sdk.ModuleSessions.beginSessionInternal(ModuleSessions.java:42) at ly.count.android.sdk.Countly.onStart(Countly.java:836) at com.company.product.messages.common.analytics.AnalyticsManagerImpl.onActivityStart(AnalyticsManager.kt:93) at com.company.product.messages.common.core.utils.MainActivityHelper$initialize$5.onStateChanged(MainActivityHelper.kt:88) at androidx.lifecycle.LifecycleRegistry$ObserverWithState.dispatchEvent(LifecycleRegistry.jvm.kt:320) at androidx.lifecycle.LifecycleRegistry.forwardPass(LifecycleRegistry.jvm.kt:257) at androidx.lifecycle.LifecycleRegistry.sync(LifecycleRegistry.jvm.kt:293) at androidx.lifecycle.LifecycleRegistry.moveToState(LifecycleRegistry.jvm.kt:142) at androidx.lifecycle.LifecycleRegistry.handleLifecycleEvent(LifecycleRegistry.jvm.kt:124) at androidx.lifecycle.ReportFragment$Companion.dispatch$lifecycle_runtime_release(ReportFragment.android.kt:190) at androidx.lifecycle.ReportFragment$LifecycleCallbacks.onActivityPostStarted(ReportFragment.android.kt:119) at android.app.Activity.dispatchActivityPostStarted(Activity.java:1476) at android.app.Activity.performStart(Activity.java:8693) at android.app.ActivityThread.handleStartActivity(ActivityThread.java:3847) at android.app.servertransaction.TransactionExecutor.performLifecycleSequence(TransactionExecutor.java:225) at android.app.servertransaction.TransactionExecutor.cycleToPath(TransactionExecutor.java:205) at android.app.servertransaction.TransactionExecutor.cycleToPath(TransactionExecutor.java:187) at android.app.servertransaction.TransactionExecutor.executeCallbacks(TransactionExecutor.java:135) at android.app.servertransaction.TransactionExecutor.execute(TransactionExecutor.java:96) at android.app.ActivityThread$H.handleMessage(ActivityThread.java:2468) at android.os.Handler.dispatchMessage(Handler.java:106) at android.os.Looper.loopOnce(Looper.java:205) at android.os.Looper.loop(Looper.java:294) at android.app.ActivityThread.main(ActivityThread.java:8248) at java.lang.reflect.Method.invoke(Native Method) at com.android.internal.os.RuntimeInit$MethodAndArgsCaller.run(RuntimeInit.java:552) at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:971)

[arifBurakDemiray]

Hello @gopidholakia 🙌
We are sorry to hear about this error. Could you please provide us what is the SDK version you are using and the SDK initialization and configuration code block. That will really helps us pinpoint the issue you are facing 😊

[gopidholakia]

Thanks for quick answer!

Version: 24.7.7
` override fun start(app: Application) {
Log.e(TAG, "start $app")
if (enabled) {
startCountly(app)
}
}

override fun onActivityStart(activity: Activity) {
    Log.e(TAG, "onActivityStart $activity")
    if (enabled) countly.onStart(activity)
}

private fun startCountly(app: Application) {
    try {
        countlyConfigForComapny(
            application = app,
            provAppId = provAppId,
        ) {
            val countlyConfig = it.getOrElse {
                Log.e(TAG, "Could not initialize countly config due to: $it")
                enabled = false
                return@countlyConfigForComapny
            }.apply {
                setLoggingEnabled(true)
                setViewTracking(true)
            }
            countly.init(countlyConfig)
            Log.d(TAG, "Countly started!")
            recordEvent("Analytics Started")
        }
    } catch (e: Exception) {
        enabled = false
        Log.e(TAG, "Could not start countly due to: ${e.localizedMessage}")
    }
}`

calling functions from MainActivityHelper

`activity.lifecycle.addObserver(
object : LifecycleEventObserver {
override fun onStateChanged(
source: LifecycleOwner,
event: Lifecycle.Event,
) {
when (event) {
Lifecycle.Event.ON_START -> {
analytics.onActivityStart(activity)
}

                    Lifecycle.Event.ON_STOP -> {
                        analytics.onActivityStop()
                    }

                    else -> {}
                }
            }
        },
    )`

[arifBurakDemiray]

Hello!

So I have seen that you are calling onStart manually.
if (enabled) countly.onStart(activity)

Is this an intended implementation? If not, Countly Android SDK handles the Activity Lifecycle Callbacks under the hood when Application class is provided in the configuration. So, you do not need to call it by hand.

And this situation is same for the lifecycle observer implementation too.

[gopidholakia]

@arifBurakDemiray where in doc it say we should need to add call onStart(this) after init.

https://support.countly.com/hc/en-us/articles/360037754031-Android#h_01HAVQDM5SXG0Q4MRTDYXJRA8H.

so we do not need to call countly.onStart(activity) manually? it will be handled automatically for all Activity we have in out app?

[arifBurakDemiray]

We are sorry that documentation is not in the latest form @gopidholakia 🙇

Yes, if application class is provided in the configuration, Countly Android SDK will handle all onStart and onStop calls of the Activities of the App.

Please have a look at our example app's App class
https://github.com/Countly/countly-sdk-android/blob/master/app/src/main/java/ly/count/android/demo/App.java

And MainActivity class,
https://github.com/Countly/countly-sdk-android/blob/master/app/src/main/java/ly/count/android/demo/MainActivity.java

Again, we are sorry for the documentation is not in the latest form. We will update it as soon as possible.
Thank you so much for your understanding 🙌

Happy coding 👋

[gopidholakia]

@arifBurakDemiray Thanks a lot for sharing insight and very quick answer!

[almozavr]

Hey, we see the same error once we enable StrictMode policies and also we don't invoke onStart manually. We provide app's context and it seems like in this scenario the Countly SDK uses exactly Application context, not Activity's one.

...
ly.count.android.sdk.DeviceInfo.getMetrics(DeviceInfo.java:639)
                 	at ly.count.android.sdk.ModuleSessions.beginSessionInternal(ModuleSessions.java:56)
                 	at ly.count.android.sdk.Countly.onStartInternal(Countly.java:922)
                 	at ly.count.android.sdk.Countly$6.onActivityStarted(Countly.java:718)
                 	at android.app.Application.dispatchActivityStarted(Application.java:401)
                 	at android.app.Activity.dispatchActivityStarted(Activity.java:1572)
                 	at android.app.Activity.onStart(Activity.java:2111)

[arifBurakDemiray]

Hello @almozavr,

Could you please provide us what is the SDK version you are using and the SDK initialization and configuration code block. That will really helps us pinpoint the issue you are facing 😊.

And to clarify, providing the "Application class" is the best way to use the SDK.

[almozavr]

@arifBurakDemiray hey 👋

we are currently on ly.count.android:sdk:24.7.8

We initialize the SDK dynamically (cause we have different urls and other parameters regarding conditions) but basically right after the app comes to foreground. The config is like:

CountlyConfig().apply {
        setApplication(applicationContext as Application)
        setContext(applicationContext)
        setServerURL("https://some-url-path")
        // setting the auth params here as well..
        enableTemporaryDeviceIdMode()
}

[almozavr]

Also, we see the same error log on init call (tried on 25.1.1)

at android.app.ContextImpl.getSystemService(ContextImpl.java:2236)
at android.content.ContextWrapper.getSystemService(ContextWrapper.java:936)
at ly.count.android.sdk.DeviceInfo$1.getDisplayMetrics(DeviceInfo.java:140)
at ly.count.android.sdk.DeviceInfo$1.getResolution(DeviceInfo.java:124)
at ly.count.android.sdk.DeviceInfo.getCommonMetrics(DeviceInfo.java:586)
at ly.count.android.sdk.DeviceInfo.getMetrics(DeviceInfo.java:639)
at ly.count.android.sdk.ModuleSessions.beginSessionInternal(ModuleSessions.java:56)
at ly.count.android.sdk.ModuleSessions.initFinished(ModuleSessions.java:168)
at ly.count.android.sdk.Countly.init(Countly.java:803)

[arifBurakDemiray]

Hello @almozavr,

I noticed that setContext is being used, but since setApplication is already handling the setup, the setContext call is unnecessary.

Additionally, for better clarity and correctness, it would be ideal to provide the Application class directly rather than casting ApplicationContext to Application.

Please let me know if you have any questions or if further clarification is needed. Thanks! 😊

[almozavr]

@arifBurakDemiray I've just tried -> it has the same outcome. It's clear why from this stack: #452 (comment) – LSSH: the SDK tries to get the display metrics from the provided application (and probably should delay this till gets activity instance instead)

[arifBurakDemiray]

Hello @almozavr,

Sorry for the delay. Could you let me know where the SDK is initialized? Is it in MainActivity or at the application level? If it's in the MainActivity, could you try moving it to Application class and see if that resolves the issue?

And could you please provide what is your development setup Android version etc. a bare minimum project. So that I could also try because I cannot reproduce the error.

This really helps us to pinpoint the issue and make the process faster
Thank you so much for your understanding 🙌