Skip to content

Align the default user-id between the C# and native layers #606

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

Closed
bruno-garcia opened this issue Mar 2, 2022 · 3 comments · Fixed by #728
Closed

Align the default user-id between the C# and native layers #606

bruno-garcia opened this issue Mar 2, 2022 · 3 comments · Fixed by #728
Assignees

Comments

@bruno-garcia
Copy link
Member

bruno-garcia commented Mar 2, 2022

SDKs usually have some default 'installation-id' as a User. When capturing from C# and Native those don't match.

Relates to: getsentry/sentry-cocoa#1554

@bruno-garcia
Copy link
Member Author

API on cocoa added: getsentry/sentry-cocoa#1680

@bruno-garcia
Copy link
Member Author

bruno-garcia commented Mar 7, 2022

@marandaneto is there a way to get the default/built-in installation-id from Android?
Do we align those already between Java/NDK layers?

Now I wonder if we're using this as the Id on cocoa or it's a different one. Installation-Id we use for did in Sessions, right? @philipphofmann might know

@marandaneto
Copy link
Contributor

NDK events don't set the user id, so the Android layer does it, you can get it via Installation.id(context), but it's package-private, if you need to access it, we need to make it public.
iOS just changed that and it has to be fixed on Flutter and React Native getsentry/sentry-cocoa#1680

@bitsandfoxes bitsandfoxes moved this from Needs Discussion to Backlog in [DEPRECATED] Mobile SDKs Apr 26, 2022
@vaind vaind self-assigned this May 4, 2022
@vaind vaind moved this from Backlog to In Progress in [DEPRECATED] Mobile SDKs May 4, 2022
@vaind vaind moved this from In Progress to Needs Review in [DEPRECATED] Mobile SDKs May 12, 2022
Repository owner moved this from Needs Review to Done in [DEPRECATED] Mobile SDKs May 16, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Archived in project
Development

Successfully merging a pull request may close this issue.

4 participants