Conversation
|
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #762 +/- ##
=======================================
Coverage 33.03% 33.03%
=======================================
Files 93 93
Lines 5788 5788
Branches 118 118
=======================================
Hits 1912 1912
Misses 3876 3876 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
There was a problem hiding this comment.
Copilot reviewed 1 out of 4 changed files in this pull request and generated no comments.
Files not reviewed (3)
- packages/website/public/install.sh: Language not supported
- packages/website/src/content/docs/setup/django.mdx: Language not supported
- packages/website/src/content/docs/setup/index.mdx: Language not supported
|
|
||
| ## 2. Enable Spotlight in your env | ||
|
|
||
| Set `$SENTRY_SPOTLIGHT` to `1` in your env |
There was a problem hiding this comment.
Add note/link to docker compose instructions here and mention this should be set to the service name.
There was a problem hiding this comment.
Also mention that for Spotlight to kick in you need settings.DEBUG to be True in Django, and no DSN is set in sentry_sdk.init() call. This is how we detect "dev mode" and is for the safety of prod environments.
There was a problem hiding this comment.
Correction, we have no limitation on DSN: https://github.com/getsentry/sentry-python/blob/master/sentry_sdk/spotlight.py
|
Why not only DEBUG? In case someone has a DSN in dev
Sent from my phone. Pardon the typos.
…On Mon, Apr 14, 2025 at 5:13 PM Burak Yigit Kaya ***@***.***> wrote:
***@***.**** commented on this pull request.
------------------------------
In packages/website/src/content/docs/setup/django.mdx
<#762 (comment)>:
> + </TabItem>
+ <TabItem label="docker">
+ ```shell
+ docker run --rm -p 8969 ghcr.io/getsentry/spotlight:latest
+ ```
+ </TabItem>
+ <TabItem label="binary">
+ ```shell
+ curl -q https://spotlightjs.com/install.sh | sh
+ ```
+ </TabItem>
+</Tabs>
+
+## 2. Enable Spotlight in your env
+
+Set `$SENTRY_SPOTLIGHT` to `1` in your env
Also mention that for Spotlight to kick in you need settings.DEBUG to be
True in Django, and no DSN is set in sentry_sdk.init() call. This is how
we detect "dev mode" and is for the safety of prod environments.
—
Reply to this email directly, view it on GitHub
<#762 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AAAFYOU6STP2LKK6W372CLL2ZQQHHAVCNFSM6AAAAAB2OZLVI2VHI2DSMVQWIX3LMV43YUDVNRWFEZLROVSXG5CSMV3GSZLXHMZDONRVGY4TKNJSGA>
.
You are receiving this because your review was requested.Message ID:
***@***.***>
|
|
@dcramer responded inline. You're right and in fact I misremembered: no DSN-related requirement. Just DEBUG |
No description provided.