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

Add support for Sentry release ENV #2

Merged
merged 3 commits into from
Sep 29, 2022
Merged

Conversation

alekseyp
Copy link
Contributor

Add support to pass Sentry release as ENV variable

@bennothommo
Copy link
Member

@alekseyp is the SENTRY_RELEASE environment variable always specified? We may have to comment it out and leave it as an example, like the original comment.

@alekseyp
Copy link
Contributor Author

alekseyp commented Sep 29, 2022

@alekseyp is the SENTRY_RELEASE environment variable always specified? We may have to comment it out and leave it as an example, like the original comment.

It's exactly the same as any other env set in this config. All of them are not always specified.

Inside sentry/sentry-laravel it's then simply checks if it's set or not.

For example:

                        if (!empty($this->release) && !$event->getRelease()) {
                            $event->setRelease($this->release);
                        }

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants