-
Notifications
You must be signed in to change notification settings - Fork 787
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.
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
🐛 BUG: Better Error Message for when Wrangler Login Fails when using Cloudflare Warp #850
Comments
threepointone
added a commit
that referenced
this issue
May 11, 2022
Using wrangler with Cloudflare WARP (https://developers.cloudflare.com/cloudflare-one/connections/connect-devices/warp/) requires using the Cloudflare certificate. This patch simply uses the certificate as NODE_EXTRA_CA_CERTS when we start wrangler. Test plan: - Turn on Cloudflare WARP/ Gateway with WARP - `wrangler dev` - Turn on Cloudflare WARP/ Gateway with DoH - `wrangler dev` - Turn off Cloudflare WARP - `wrangler dev` Fixes #953, #850
threepointone
added a commit
that referenced
this issue
May 11, 2022
Using wrangler with Cloudflare WARP (https://developers.cloudflare.com/cloudflare-one/connections/connect-devices/warp/) requires using the Cloudflare certificate. This patch simply uses the certificate as NODE_EXTRA_CA_CERTS when we start wrangler. Test plan: - Turn on Cloudflare WARP/ Gateway with WARP - `wrangler dev` - Turn on Cloudflare WARP/ Gateway with DoH - `wrangler dev` - Turn off Cloudflare WARP - `wrangler dev` Fixes #953, #850
threepointone
added a commit
that referenced
this issue
May 11, 2022
Using wrangler with Cloudflare WARP (https://developers.cloudflare.com/cloudflare-one/connections/connect-devices/warp/) requires using the Cloudflare certificate. This patch simply uses the certificate as NODE_EXTRA_CA_CERTS when we start wrangler. Test plan: - Turn on Cloudflare WARP/ Gateway with WARP - `wrangler dev` - Turn on Cloudflare WARP/ Gateway with DoH - `wrangler dev` - Turn off Cloudflare WARP - `wrangler dev` Fixes #953, #850
Merged
Fixed properly in #963, we don't need to have an error anymore |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
What version of
Wrangler
are you using?0.0.19
What operating system are you using?
Mac
Describe the Bug
I tried running the
wrangler login
command but it timed out with this error message.I was able to fix it by turning off WARP and retying the login process and it worked. We should include a better error message instead that tell me what the issue is and catch the error returned.
The text was updated successfully, but these errors were encountered: