getsentry / getsentry/sentry-javascript

feat(browser): Add environment variable support for Spotlight configuration

未关闭
#18,404 1 条评论 0 个 reaction 已指派 1 人 已被 @BYK 认领 在 GitHub 查看
Feature
主要语言
TypeScript
星标
8.7k
派生
1.8k
平均合并
1 天 17 小时
30 天内合并 PR
515

描述

> \[!NOTE\]
The pull request "[feat(browser): Add environment variable support for Spotlight configuration]()" was created by @BYK but did not reference an issue. Therefore this issue was created for better visibility in external tools like Linear.

Implements full Spotlight spec with support for multiple framework-specific
environment variable prefixes. Adds defensive environment variable access
for both process.env and import.meta.env to support various bundlers.

Supported environment variables (in priority order):

* `PUBLIC_SENTRY_SPOTLIGHT` (SvelteKit, Astro, Qwik)
* `NEXT_PUBLIC_SENTRY_SPOTLIGHT` (Next.js)
* `VITE_SENTRY_SPOTLIGHT `(Vite)
* `NUXT_PUBLIC_SENTRY_SPOTLIGHT `(Nuxt)
* `REACT_APP_SENTRY_SPOTLIGHT `(Create React App)
* `VUE_APP_SENTRY_SPOTLIGHT `(Vue CLI)
* `GATSBY_SENTRY_SPOTLIGHT `(Gatsby)
* `SENTRY_SPOTLIGHT` (base/official)

`SENTRY_SPOTLIGHT` is last as in environments like Docker Compose, we actually make the front-end env variable different than the base `SENTRY_SPOTLIGHT` one -- the backends need to reach `docker.host.internal` whereas front-ends always need `localhost` as we assume the browser runs on the same host with Spotlight.

Refactors envToBool utility from node-core to core package for shared usage.
Adds resolveSpotlightOptions utility to ensure consistent precedence rules
across Browser and Node SDKs.

Includes comprehensive test coverage for all new utilities and integration
tests for environment variable precedence behavior.

贡献指南

打开贡献指南

评估

这个 Issue 还没有评估数据。

把新 issue 发到你的邮箱

精选适合新手参与的 GitHub issue 摘要。