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

Does not report crashes #447

Closed
vovanmix opened this issue Jul 25, 2018 · 15 comments
Closed

Does not report crashes #447

vovanmix opened this issue Jul 25, 2018 · 15 comments

Comments

@vovanmix
Copy link

vovanmix commented Jul 25, 2018

OS:

  • MacOS

Platform:

  • iOS

Output of node -v && npm -v && npm ls --prod --depth=0

v8.10.0
5.6.0
/Users/xxx/projects/yyy
├── apollo-boost@0.1.6
├── aws-sdk@2.273.1
├── babel-cli@6.26.0
├── babel-eslint@8.2.3
├── babel-plugin-syntax-trailing-function-commas@6.22.0
├── babel-plugin-transform-class-constructor-call@6.24.1
├── babel-plugin-transform-do-expressions@6.22.0
├── babel-plugin-transform-exponentiation-operator@6.24.1
├── babel-plugin-transform-export-extensions@6.22.0
├── babel-plugin-transform-function-bind@6.22.0
├── babel-plugin-transform-inline-environment-variables@0.4.3
├── babel-preset-env@1.7.0
├── babel-preset-expo@4.0.0
├── babel-preset-flow@6.23.0
├── date-fns@1.29.0
├── eslint@4.19.1
├── eslint-config-airbnb@16.1.0
├── eslint-config-prettier@2.9.0
├── eslint-plugin-class-property@1.1.0
├── eslint-plugin-flowtype@2.46.3
├── eslint-plugin-import@2.11.0
├── eslint-plugin-jsx-a11y@6.0.3
├── eslint-plugin-prettier@2.6.0
├── eslint-plugin-react@7.7.0
├── ex-react-native-i18n@0.0.4
├── expo@28.0.0
├── flow-bin@0.67.1
├── formik@0.11.11
├── graphql@0.13.2
├── graphql-tag@2.9.2
├── jest-expo@25.0.0
├── lodash@4.17.10
├── prettier@1.13.0
├── react@16.3.1
├── react-apollo@2.1.4
├── react-native@0.55.4 invalid
├── react-native-actionsheet@2.4.2
├── react-native-iap@1.3.1
├── react-native-image-pan-zoom@2.1.5
├── react-native-mime-types@2.2.1
├── react-native-phone-call@1.0.6
├── react-native-scripts@1.13.1
├── react-native-sentry@0.38.2
├── react-native-swiper-flatlist@1.0.7
├── react-navigation@2.4.1
├── react-test-renderer@16.2.0
├── rn-sliding-up-panel@1.2.1
└── yup@0.25.1

Config:

Sentry.config('https://...@sentry.io/...').install()

I have following issue:

Native crashes are not reported. I have a detached expo app, I'm using react-native-sentry instead of sentry-expo because it's detached, I did react-native link react-native-sentry. Running the app from expo.
When the app crashes on iOS, or when I do Sentry.nativeCrash(); in the simulator, nothing gets reported. Sentry.captureMessage('TEST message'); reports correctly.
It also wan't reporting crashes before I detached expo, with sentry-expo.

@sergeylaptev
Copy link

sergeylaptev commented Jul 28, 2018

Confirm, the same situation. Installed the latest release (0.38.3). Crashes don't send but custom captured messages are shown in the log.
Sentry.isNativeClientAvailable() returns false for Android

@HazAT
Copy link
Member

HazAT commented Jul 30, 2018

I just tried it and it works.
Please make sure your MainApplication.java looks something like this:

...
import io.sentry.RNSentryPackage;
...
public class MainApplication extends Application implements ReactApplication {
...
    @Override
    protected List<ReactPackage> getPackages() {
      return Arrays.<ReactPackage>asList(
          new MainReactPackage(), new RNSentryPackage()
      );
    }

@vovanmix
Copy link
Author

vovanmix commented Jul 30, 2018 via email

@sergeylaptev
Copy link

@HazAT Mine looks exactly the same, even built-in native crash method doesn't trigger error event in the dashboard (crashes the app though)

@HazAT
Copy link
Member

HazAT commented Jul 30, 2018

What is the SDK section saying on the bottom of your event?

It should look something like this:
screenshot 2018-07-30 15 36 53

@idris
Copy link

idris commented Jan 13, 2019

@vovanmix @sergeylaptev did you ever figure this out? We don't see any crashes in Segment, though JS exceptions and messages work fine.

@MorcosS
Copy link

MorcosS commented Mar 28, 2019

Any news about this? I am trying Sentry on both ios and android it triggers only when using Sentry.captureException().
I am using this with:
"react-native": "0.59.0", "react-native-sentry": "^0.42.0",

@Sam-Hoult
Copy link

Getting reports from customers of the app crashing and we see nothing on our end.
To make the app crash show up, but we aren't getting anything when the app just drops.

@NetPumi2
Copy link

+1

@cinder92
Copy link

cinder92 commented May 3, 2019

same issue here...

@zivchen
Copy link

zivchen commented May 26, 2019

Same here
"react-native": "0.59.8",
"react-native-sentry": "^0.42.0",

@509dave16
Copy link

+1

@prof18
Copy link

prof18 commented Jun 14, 2019

Same here. As workaround I override the global error handler to include the Sentry log.

// Error handler setup
  let originalHandler = ErrorUtils.getGlobalHandler();
  ErrorUtils.setGlobalHandler((error, isFatal) => {
    Sentry.captureException(error);
    originalHandler(error, isFatal);
  });

@terreb
Copy link

terreb commented Jun 25, 2019

@prof18, not working for me. react-native: 0.57.8, react-native-sentry: 0.43.2

EDIT: sorry, my bad, I tested with the debug build where Sentry was disabled. It does work. Thank you!

@HazAT
Copy link
Member

HazAT commented Sep 4, 2019

I am closing all old issues, please if this is still a problem feel free to revive it.
Also, consider moving to our new SDK @sentry/react-native if it still happens there please open a new issue.

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

No branches or pull requests