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

IllegalArgumentExceptionion: io.sentry.event.Breadcrumb one of 'message' or 'data' must be set #623

Closed
2 of 5 tasks
spruce-bruce opened this issue Jul 16, 2019 · 0 comments · Fixed by #749
Closed
2 of 5 tasks

Comments

@spruce-bruce
Copy link

spruce-bruce commented Jul 16, 2019

OS:

  • Windows
  • MacOS
  • Linux

Platform:

  • iOS
  • Android

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

aaron$ node -v && npm -v && npm ls --prod --depth=0
v11.10.1
6.7.0
HeliaConnect@1.15.0 /Users/aaron/code/synapse/helia/HeliaConnect
├── @redux-offline/redux-offline@2.5.1
├── @sentry/cli@1.31.0
├── @synapsestudios/fetch-client@2.0.0-rc2
├── @synapsestudios/fetch-client-goalie@1.0.1
├── appcenter@1.9.0
├── appcenter-analytics@1.9.0
├── appcenter-crashes@1.9.0
├── async@2.6.2
├── date-fns@1.30.1
├── formik@1.5.0
├── immutable@3.8.2
├── jest-serializer@23.0.1
├── lodash@4.17.11
├── numeral@2.0.6
├── prop-types@15.7.2
├── querystring@0.2.0
├── raven-for-redux@1.4.0
├── raven-js@3.27.0
├── UNMET PEER DEPENDENCY react@16.8.2
├── react-immutable-proptypes@2.1.0
├── react-native@0.57.4
├── react-native-actionsheet@2.4.2
├── react-native-android-open-settings@1.3.0
├── react-native-blur@3.2.2
├── react-native-camera@1.3.1
├── react-native-datepicker@1.7.2
├── react-native-fast-image@5.1.4
├── react-native-file-viewer@1.0.13
├── react-native-firebase@5.1.0
├── react-native-fs@2.13.3
├── react-native-image-resizer@1.0.0
├── react-native-keyboard-aware-scroll-view@0.4.4
├── react-native-keychain@3.0.0
├── react-native-masked-text@1.10.1
├── react-native-modal@4.1.1
├── react-native-modalbox@1.7.1
├── react-native-permissions@1.1.1
├── react-native-segmented-control-tab@3.4.0
├── react-native-sentry@0.37.0
├── react-native-signature-capture@0.4.9
├── react-native-sqlite-storage@3.3.9
├── react-native-svg@8.0.9
├── react-native-swipeable@0.6.0
├── react-native-touch-id@4.3.0
├── react-native-version-number@0.3.5
├── react-native-zip-archive@2.2.6
├── react-navigation@2.18.3
├── react-redux@5.1.1
├── redux@3.7.2
├── redux-persist@4.10.2
├── redux-persist-transform-immutable@5.0.0
├── redux-thunk@2.3.0
├── reselect@3.0.1
├── timezonecomplete@5.7.0
├── tzdata@1.0.15
├── url-search-params-polyfill@2.0.3
├── uuid@3.3.2
└── yup@0.24.1

Config:

Sentry.config('https://...@sentry.io/...').install();
Sentry.setTagsContext({
  environment: config.env,
  react: true
});

I have following issue:

When android http client fails due to an unexpected end of socket error sentry throws an error "one of 'message' or 'data' must be set". The end of socket error is described here: https://codeday.me/en/qa/20190305/2222.html

Steps to reproduce:

  • Using a tool like charles proxy, throttle your connection. It's important to use a tool that allows you to set "reliability" low so that you get failing network requests. This error can occur without using a tool to disrupt your network requests, but it's incredibly hard to reproduce in the wild unless you have a readily available unreliable network.
  • Make network requests using fetch
  • Occasionally a network failure will cause the error described in this article: https://codeday.me/en/qa/20190305/2222.html
  • When that error occurs, sentry throws the error "IllegalArgumentExceptionion: io.sentry.event.Breadcrumb one of 'message' or 'data' must be set"
  • javascript execution is stopped at this point

Actual result:

  • sentry illegal argument exception is thrown, js execution is stopped

Expected result:

  • don't throw that error. im not sure if js execution is stopped because of the sentry error or the java error, but i'd hope to be able to catch the error and respond gracefully
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

Successfully merging a pull request may close this issue.

1 participant