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

SDKs should not send tags by default but rather index them in snuba if it needs to be searchable #2444

Open
Tracked by #60
markushi opened this issue Dec 28, 2022 · 2 comments

Comments

@markushi
Copy link
Member

markushi commented Dec 28, 2022

Description

We should avoid setting tags when possible.

See meta issue for more details: getsentry/team-mobile#60

I just checked Java/Android usage:

  • Any tags set via SentryOptions.addContextTag() - but since they're custom values and empty by default I guess we can ignore those
  • setTag("TimberTag", ...) -> link
  • setSideLoadedInfo(...) -> link
@kahest
Copy link
Member

kahest commented Jan 11, 2023

Need to talk to SnS to find out if using Tags makes sense in this case, and to make sure functionality is not lost if moved to Context.

@marandaneto
Copy link
Contributor

Per getsentry/team-mobile#60 (comment)
This issue can also be closed unless there are something new after my discussion with S&S

@markushi markushi removed this from the 7.0.0 milestone May 31, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: Backlog
Development

No branches or pull requests

4 participants