Skip to content
This repository has been archived by the owner on Aug 3, 2023. It is now read-only.

dont eat errors #2022

Merged
merged 1 commit into from
Aug 9, 2021
Merged

dont eat errors #2022

merged 1 commit into from
Aug 9, 2021

Conversation

caass
Copy link
Contributor

@caass caass commented Aug 9, 2021

Print the original error when we create an error report.

This should help in debugging install errors, and other stuff where we cant report it
due to wrangler not being installed correctly

@caass caass requested a review from a team as a code owner August 9, 2021 13:54
@caass caass requested a review from nilslice August 9, 2021 13:54
Copy link
Contributor

@jyn514 jyn514 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The new output looks like this:

Oops! Wrangler encountered an error:

panicked at 'oops', src/main.rs:38:5

Please help Cloudflare debug this issue by submitting the error report
(/home/jnelson/.wrangler/errors/1628522085430.log)

To submit this error report to Cloudflare, run:

    $ wrangler report
        
Aborted (core dumped)

@caass caass merged commit 8140967 into master Aug 9, 2021
@delete-merged-branch delete-merged-branch bot deleted the cass/dont-eat-error branch August 9, 2021 15:58
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants