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

Crash on reportSoftException when lineNumber is missing #10

Closed
yogeshchoudhary147 opened this issue Apr 18, 2019 · 5 comments
Closed

Crash on reportSoftException when lineNumber is missing #10

yogeshchoudhary147 opened this issue Apr 18, 2019 · 5 comments

Comments

@yogeshchoudhary147
Copy link

Android app crashes when ExceptionsManagerModule.reportSoftException is called with a stack trace that has frames without lineNumber.

The following fatal exception is logged.

2019-04-09 16:14:29.742 7083-7781/? E/AndroidRuntime: FATAL EXCEPTION: mqt_native_modules Process: co.armortext.messenger, PID: 7083 com.facebook.react.bridge.NoSuchKeyException: lineNumber at com.facebook.react.bridge.ReadableNativeMap.getValue(ReadableNativeMap.java:124) at com.facebook.react.bridge.ReadableNativeMap.getValue(ReadableNativeMap.java:128) at com.facebook.react.bridge.ReadableNativeMap.getInt(ReadableNativeMap.java:182) at com.facebook.react.util.JSStackTrace.format(JSStackTrace.java:28) at com.facebook.react.modules.core.ExceptionsManagerModule.reportSoftException(ExceptionsManagerModule.java:46) at java.lang.reflect.Method.invoke(Native Method) at com.facebook.react.bridge.JavaMethodWrapper.invoke(JavaMethodWrapper.java:372) at com.facebook.react.bridge.JavaModuleWrapper.invoke(JavaModuleWrapper.java:158) at com.facebook.react.bridge.queue.NativeRunnable.run(Native Method) at android.os.Handler.handleCallback(Handler.java:789) at android.os.Handler.dispatchMessage(Handler.java:98) at com.facebook.react.bridge.queue.MessageQueueThreadHandler.dispatchMessage(MessageQueueThreadHandler.java:29) at android.os.Looper.loop(Looper.java:164) at com.facebook.react.bridge.queue.MessageQueueThreadImpl$4.run(MessageQueueThreadImpl.java:232) at java.lang.Thread.run(Thread.java:764)

It started on 5th April when 0.1.5 was released.

I can confirm that downgrading it to 0.1.4, be specifying it in package.json, has resolved the issue for us.

@Calamari
Copy link
Contributor

Calamari commented Apr 29, 2019

That looks like a Java stack trace. I never actively tested any stack traces except JavaScript so far. So there was no guarantee that it would work. But I can check it out and see what happens, but for that, it might be helpful to get the original stack trace, on which the parsers choke. Can you access it somehow?

@Calamari
Copy link
Contributor

I just realized that this is an error happening within react bridge. Is it possible to get the original stack trace, that causes that exception, @yogeshchoudhary147?

@hfrosh
Copy link

hfrosh commented Apr 30, 2019

@Calamari I believe this this bug was posted as a result of this issue reported for React Native: facebook/react-native#24382

@dimlksin2
Copy link

com.facebook.react.bridge.NoSuchKeyException: lineNumber
at com.facebook.react.bridge.ReadableNativeMap.getValue(ReadableNativeMap.java:122)
at com.facebook.react.bridge.ReadableNativeMap.getInt(ReadableNativeMap.java:158)
at com.facebook.react.util.JSStackTrace.format(JSStackTrace.java:28)
at com.facebook.react.modules.core.ExceptionsManagerModule.reportSoftException(ExceptionsManagerModule.java:46)
at java.lang.reflect.Method.invoke(Native Method)
at com.facebook.react.bridge.JavaMethodWrapper.invoke(JavaMethodWrapper.java:372)
at com.facebook.react.bridge.JavaModuleWrapper.invoke(JavaModuleWrapper.java:160)
at com.facebook.react.bridge.queue.NativeRunnable.run(Native Method)
at android.os.Handler.handleCallback(Handler.java:751)
at android.os.Handler.dispatchMessage(Handler.java:95)
at com.facebook.react.bridge.queue.MessageQueueThreadHandler.dispatchMessage(MessageQueueThreadHandler.java:29)
at android.os.Looper.loop(Looper.java:154)
at com.facebook.react.bridge.queue.MessageQueueThreadImpl$3.run(MessageQueueThreadImpl.java:192)
at java.lang.Thread.run(Thread.java:762)

@Calamari
Copy link
Contributor

Calamari commented Sep 9, 2019

Since this seems not to be a bug in this library, rather in the usage within the Java react-bridge app, I will close this.
But I regularly monitor what happens within the react-native discussion.

@Calamari Calamari closed this as completed Sep 9, 2019
telldus-bot pushed a commit to telldus/telldus-live-mobile-v3 that referenced this issue Aug 5, 2020
See if crashes occur again with the latest version
errwischt/stacktrace-parser#10
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

4 participants