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

Add event tags so users can easily determine and filter where events originate from. #890

Merged
merged 5 commits into from
Jun 2, 2020

Conversation

jennmueng
Copy link
Member

📢 Type of change

  • Bugfix
  • New feature
  • Enhancement
  • Refactoring

📜 Description

Added a new tag event.origin to events with values

  • javascript
  • ios
  • android

Along with an event.environment tag for native android events with values:

  • java
  • native

These tags are attached onto the event in the beforeSend function in both iOS and Android and determined using the SDK name. On javascript, the event is added to the scope on init.

💡 Motivation and Context

This change allows users to easily determine, search, and filter where the event originated in react native apps with native code running alongside javascript.

We use the simplified tag values above instead of the SDK names such as sentry.java.android or sentry.javascript.react-native as the goal is to make it easy to search and filter for the end-user.

💚 How did you test it?

Tested using the sample app by testing all the options, and confirmed that the tags log correctly on the Sentry Test Dashboard. All existing tests are passing.

📝 Checklist

  • I reviewed submitted code
  • I added tests to verify changes
  • All tests passing

🔮 Next steps

jennmueng added 4 commits May 29, 2020 22:22
The event.origin tag is added to the RN SDK Scope on init.
On Android, we have a switch
in the native beforeSend that determines where the event originated to set
the tag.
On iOS, the 'ios' tag is added in the native beforeSend if the event
originated in the sentry-cocoa sdk

Addresses #881
Copy link
Member

@HazAT HazAT left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🥇

@jennmueng jennmueng merged commit 9a72f99 into master Jun 2, 2020
@jennmueng jennmueng deleted the feat/event-origin-tag branch June 2, 2020 08:59
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 this pull request may close these issues.

2 participants