getsentry / getsentry/sentry-dotnet
device context not included in new sentry .net sdk
- Dominant language
- C#
- Stars
- 770
- Forks
- 248
- Avg merge
- 3d 4h
- Merged PRs (30d)
- 49
Description
When attempting to integrate with sentry .net using the nlog integration, I noticed that device context wasn't being sent up like other platforms already do.
The older raven-csharp sdk appears to support adding this information based on https://github.com/getsentry/raven-csharp/blob/4fdc94eb514d96c22a83d851e2141fa9df10f613/src/app/SharpRaven/Data/Context/Device.cs#L212. (I haven't actually integrated with raven-csharp to verify, just assuming based on this file).
Is sentry-dotnet intended to not add this information by default? Example information I'm looking for is cpu arch, screen_resolution, storage, memory, and etc
## TLDR;
> These folks are adding additional context to the SDK based on some other APIs available. We could learn something from them:
>
> https://github.com/Valkirie/HandheldCompanion/blob/32e011cc677312e3d6894a7fb758924ee6bbeafb/HandheldCompanion/Devices/IDevice.cs#L560-L567
Contributor guide
Assessment
This issue has not been assessed yet.