Read DSN, Environment and Release options from environment variables
#1054
Conversation
Instructions and example for changelogPlease add an entry to Example: ## Unreleased
- Read `DSN`, `Environment` and `Release` options from environment variables ([#1054](https://github.com/getsentry/sentry-unreal/pull/1054))If none of the above apply, you can opt out of this check by adding |
![cursor[bot]](https://avatars.githubusercontent.com/in/1210556?s=60&v=4){kind=small}
This comment was marked as outdated.
This comment was marked as outdated.
Sorry, something went wrong.
| { | ||
| FormattedReleaseName = FString::Printf(TEXT("%s@%s"), *FormattedReleaseName, *Version); | ||
| UE_LOG(LogSentrySdk, Log, TEXT("DSN is not set in plugin settings - using SENTRY_DSN environment variable instead.")); |
There was a problem hiding this comment.
Note: Since Engine config is still priority, we will need to adapt demo when we update the SDK version there.
| return EnvVarEnvironment; | ||
| } | ||
|
|
||
| UE_LOG(LogSentrySdk, Log, TEXT("Using current build configuration as Sentry environment.")); |
There was a problem hiding this comment.
Do we also want to add a log message when using the environment from engine config? At least verbose. Same for GetEffectiveRelease()
There was a problem hiding this comment.
Added a verbose log message when the value from the plugin settings is used (for both release and environment).
| */ | ||
| FString GetEffectiveEnvironment() const; | ||
|
|
||
| FString GetEnvironmentFromBuildConfig() const; |
There was a problem hiding this comment.
Let's also add documentation for the two other functions we added here
This PR adds support for configuring the plugin’s DSN, Environment, and Release options via environment variables when they are not set explicitly (useful for CI workflows).
Related docs update:
Closes #1044