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

[dev] Improves error message when there is no internet connection #1114

Merged
merged 3 commits into from
Mar 3, 2020

Conversation

EverlastingBugstopper
Copy link
Contributor

Fixes #1093

Before:

Error: error sending request for url (https://api.cloudflare.com/client/v4/accounts/12s4t3i5n8k8y0f2ar7t5s1/workers/scripts/worker/preview): error trying to connect: dns error: failed to lookup address information: nodename nor servname provided, or not known

After:

Error: Could not upload your script, check your internet connection or https://www.cloudflarestatus.com/ for any incidents impacting the Cloudflare Workers API.

src/commands/dev/mod.rs Outdated Show resolved Hide resolved
Co-Authored-By: Victoria Bernard <bernard.victoria22@gmail.com>
@EverlastingBugstopper EverlastingBugstopper mentioned this pull request Mar 3, 2020
@ashleymichal
Copy link
Contributor

nit: in theory you should be able to see the difference between a network call that never hit the server and one that came back with a gateway error.

@EverlastingBugstopper EverlastingBugstopper merged commit 3651603 into master Mar 3, 2020
@delete-merged-branch delete-merged-branch bot deleted the avery/no-internet-msg branch March 3, 2020 20:42
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.

1.8.0-rc.1 needs internet to build locally (-dev)
3 participants