Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

NPE at com.microsoft.identity.common.internal.ui.browser.BrowserSelector.isFullBrowser(BrowserSelector.java:236) #2084

Open
zeeshanjamal opened this issue Jun 20, 2023 · 2 comments

Comments

@zeeshanjamal
Copy link

zeeshanjamal commented Jun 20, 2023

I am integrating Intune SDK(with MSAL) on my android application. I am able to login via MSAL and am also seeing the Microsoft Intune's consent screen where i have to specify the pin. After both these steps application crashes with the exception trace below. I am using the latest Intune SDK(9.7.0), android api version is 33 and MSAL is 4.6.1. Due to this i am not getting
ENROLLMENT_SUCCEEDED intune notification at the end. I am using Capacitor with react.

at com.microsoft.identity.common.internal.ui.browser.BrowserSelector.isFullBrowser(BrowserSelector.java:236)
at com.microsoft.identity.common.internal.ui.browser.BrowserSelector.getBrowsers(BrowserSelector.java:199)
at com.microsoft.identity.common.internal.ui.browser.BrowserSelector.getDefaultBrowser(BrowserSelector.java:144)
at com.microsoft.identity.common.internal.ui.browser.BrowserSelector.select(BrowserSelector.java:79)

@zeeshanjamal
Copy link
Author

zeeshanjamal commented Jun 20, 2023

IntentFilter is null in ResolveInfo. Same issue as mentioned in this post:
Android msal login failed

Similar issue as mentioned in below post as well but that is in Open ID App auth package of android.
NPE on isFullBrowser method . They fixed it by adding null check on Intent Filter of ResolveInfo.

@zeeshanjamal
Copy link
Author

I was able to fix it by setting "authorization_user_agent" : "WEBVIEW". Earlier it was set to "DEFAULT.
But I am still curious about a null check. Do we still need that?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant