Crash caused by java.lang.NullPointerException: dataIntent must not be null (potentially related: New chrome Version)

[KeiShadow]

Issue

Hello, my application crashed with latest chrome browser( 72.0.3626.76)
In app i detected if the chrome is installed if yes the redirect url is change for page after the correct login the ids server will redirect user to page where is a button with required params for redirect back to APP. The login worked very well since last chrome update. The another browser working good.

Log:
E/AndroidRuntime: FATAL EXCEPTION: main
Process: secret, PID: 1804
java.lang.RuntimeException: Failure delivering result ResultInfo{who=null, request=0, result=0, data=null} to activity {secret/secret.MainActivity}: java.lang.NullPointerException: dataIntent must not be null
at android.app.ActivityThread.deliverResults(ActivityThread.java:4268)

Environment

• Your Identity Provider: dentityServer 4
• Platform that you're experiencing the issue on:
  • iOS
  • Android
  • iOS but have not tested behavior on Android
  • Android but have not tested behavior on iOS
  • Both
• Are you using Expo, e.g. ExpoKit?
  • No
  • Yes, I've not ejected
  • Yes, but I have ejected to ExpoKit
  • Yes, but I have ejected to vanilla React Native

[sraka1]

Looks like this is also related to my comment in #130 and is getting more and more frequent? Maybe a newer Chrome is somehow the problem here?

@kadikraman have you had this issue occur as well?

[kadikraman]

Hi @sraka1 - I haven't had the chance to investigate this yet, but it's on my backlog.

[sraka1]

@kadikraman could you please rename this issue into "Crash caused by java.lang.NullPointerException: dataIntent must not be null". I don't think the version of Chrome (or any other browser) is to blame here...

[kadikraman]

Thanks for the tip, @sraka1 - the new Chrome version may be a coincidence in this case.

[Jay-A-McBee]

Closing this out - should be fixed by #352.