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.

Sign up for GitHub

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

Jump to bottom

[wrangler] Log Error#cause and DOMException reasons when running wrangler dev #2851

Closed
wants to merge 9 commits into from
Closed

[wrangler] Log Error#cause and DOMException reasons when running wrangler dev #2851

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
9 commits
Select commit Hold shift + click to select a range
58e32d6
Reuse `SourceMapConsumer` between source-mapping operations
mrbbot Mar 3, 2023
f77d821
Lift out source-mapping to function
mrbbot Mar 3, 2023
79863f6
Log `Error#cause` in `dev`
mrbbot Mar 3, 2023
b41f08e
Log `DOMException` causes in `dev`
mrbbot Mar 3, 2023
a04c545
Add changeset
mrbbot Mar 3, 2023
7e43a21
fixup! Log `Error#cause` in `dev`
mrbbot Mar 6, 2023
bd56f93
Fix facade source maps on macOS
mrbbot Mar 6, 2023
3a6f17c
Rename `consumer` back to `sourceMapConsumer`
mrbbot Mar 6, 2023
fac6553
Remove vendored `stack-trace` package
mrbbot Mar 6, 2023
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
13 changes: 13 additions & 0 deletions .changeset/fast-rats-turn.md
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -0,0 +1,13 @@
---
"wrangler": minor
---

feat: log error causes when running `wrangler dev`

Previously, the `cause` property of errors was ignored when logging.
This is used by D1 to include context on why an operation failed.
Because we weren't logging this, D1 failures would always show up as
`D1_ERROR`, with no additional context, making debugging difficult.

We now attempt to source-map and log these, and the reasons for
`DOMException`s.
Loading