Skip to content

fix: Always trim befor trying to JSON.parse#438

Merged
BYK merged 1 commit intomainfrom
byk/fix/fix-json-parse-empty
Jul 11, 2024
Merged

fix: Always trim befor trying to JSON.parse#438
BYK merged 1 commit intomainfrom
byk/fix/fix-json-parse-empty

Conversation

@BYK
Copy link
Member

@BYK BYK commented Jul 11, 2024

In #429, we started sending an empty payload at the start to trigger connect event on Firefox. Our data parser does not account for this and gets confused, causing an error. This patch fixes that.

In #429, we started sending an empty payload at the start to trigger connect event on Firefox. Our data parser does not account for this and gets confused, causing an error. This patch fixes that.
@vercel
Copy link

vercel bot commented Jul 11, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
spotlightjs ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jul 11, 2024 10:09pm

@BYK BYK merged commit c10fd3f into main Jul 11, 2024
@BYK BYK deleted the byk/fix/fix-json-parse-empty branch July 11, 2024 22:12
kensternberg-authentik added a commit to kensternberg-authentik/spotlight that referenced this pull request Jul 15, 2024
…rybook-8-compatibility

* origin/HEAD:
  fix: Overhaul envelope parsing to be spec compliant (getsentry#439)
  fix: Always trim befor trying to JSON.parse (getsentry#438)
  Highlighted active span and event (getsentry#437)
  ref: Use new event helpers (getsentry#433)
  fix: Remove forgotton console.log statement
  feat: POC for adding spotlight overlay for ssr-errors (getsentry#364)
  ref: More concise event helpers (getsentry#431)
  fix: Fix CI issues (getsentry#432)
  fix: Send something immediately to trigger open event (getsentry#429)
  ref: More robust client init code generation (getsentry#426)
  docs: Add note about Volta - pnpm support (getsentry#425)
  Added Search in Trace detail (getsentry#424)
  Added Trace Info (getsentry#423)
  chore(deps-dev): bump vite from 4.5.2 to 4.5.3 (getsentry#392)
  chore(deps): bump braces from 3.0.2 to 3.0.3 (getsentry#415)
  docs: Add build step to initial setup (getsentry#418)
dcramer pushed a commit that referenced this pull request Jun 3, 2025
In #429, we started sending an empty payload at the start to trigger
connect event on Firefox. Our data parser does not account for this and
gets confused, causing an error. This patch fixes that.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant