getsentry / getsentry/sentry-dotnet

Native integration: cache directory path

Open
#2,924 2 comments 0 reactions 0 assignees View on GitHub
.NET Improvement
Dominant language
C#
Stars
770
Forks
248
Avg merge
3d 4h
Merged PRs (30d)
49

Description

Native backend uses offline caching to store the error at the time of the crash and only sends it later on the second app run. what do we want to do about the cache directory path? Defaults to the current working directory. Can we figure out a more reasonable default or do we want to document this? Or even warn unless explicitly specified?

> See [https://github.com/getsentry/sentry-dotnet/pull/2887#issuecomment-1830328182]()
>
> > what do we want to do about the cache directory path? Defaults to the current working directory. Can we figure out a more reasonable default or do we want to document this? Or even warn unless explicitly specified?
>
> This is a problem on a Mac where the OS asks the user to give permission if the file is on Documents and others.
I wonder if using a consistent path name under `Path.GetTempPath` is an alternative.
>
> It's a problem on Unity today btw ^ so we could align the solution here
>
> ALso a problem the fact we don't have paths by default. And the .NET SDK doesn't support having more than 1 instance of the app running at the same time (afaik not fixed) as worker transports would race to pick work off of the directory

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.