razorpay / react-native-razorpay

React Native wrapper for Razorpay's mobile SDKs
https://www.npmjs.com/package/react-native-razorpay
MIT License
121 stars 107 forks source link

App Crashing when using getCardNetworkLength method #476

Open sudhanshu-housing opened 2 months ago

sudhanshu-housing commented 2 months ago

Description

When using getCardNetworkLength method from react-native-customui lib app crashed only once for android device(Poco X6 Pro 5G) with android version 14. Apart from this it is working fine for others.

Razorpay Package Version :

react-native-customui@2.2.4

What you did:

Got this crash on firebase crashlytics

What happened:

Fatal Exception: java.lang.NullPointerException: Attempt to invoke virtual method 'boolean org.json.JSONObject.has(java.lang.String)' on a null object reference
       at com.razorpay.RazorpayUtils.c(RazorpayUtils.java:269)
       at com.razorpay.BaseRazorpay.getCardNetworkLength(BaseRazorpay.java)
       at com.razorpay.rn.RazorpayModule.getCardNetworkLength(RazorpayModule.java:109)
       at java.lang.reflect.Method.invoke(Method.java)
       at com.facebook.react.bridge.JavaMethodWrapper.invoke(JavaMethodWrapper.java:372)
       at com.facebook.react.bridge.JavaModuleWrapper.invoke(JavaModuleWrapper.java:188)
       at com.facebook.jni.NativeRunnable.run(NativeRunnable.java)
       at android.os.Handler.handleCallback(Handler.java:958)
       at android.os.Handler.dispatchMessage(Handler.java:99)
       at com.facebook.react.bridge.queue.MessageQueueThreadHandler.dispatchMessage(MessageQueueThreadHandler.java)
       at android.os.Looper.loopOnce(Looper.java:222)
       at android.os.Looper.loop(Looper.java:314)
       at com.facebook.react.bridge.queue.MessageQueueThreadImpl$4.run(MessageQueueThreadImpl.java:228)
       at java.lang.Thread.run(Thread.java:1012)

Steps To Reproduce

Provide a detailed list of steps that reproduce the issue.

  1. Invoked getCardNetworkLength method from android device(Poco X6 Pro 5G)
  2. App crashed
vivekshindhe commented 2 months ago

@sudhanshu-housing , Is the issue reproducible on that certain device now as well? Can you please check if there are more crashes of this sort?