-
Notifications
You must be signed in to change notification settings - Fork 334
Conversation
1f5881f
to
6e6df29
Compare
So after talking about it with @EverlastingBugstopper we thought about how to structure https with dev. With dev there's the local server which accepts your requests and there's what you upstream to. We were thinking that making https default for both local and upstream would add a lot of friction to users when they run into browser warnings and curl failing without |
To clarify... Are you saying that I think that would be a dangerous default. We should be encrypting all network traffic unless the user very explicitly tells us not to. Moreover, most sites these days enable "always use HTTPS" which causes all HTTP requests to return a redirect to HTTPS, which would likely lead to confusion when using I do agree that traffic to |
That's what I was intending but this makes sense. Local http and upstream https by default sound good? @EverlastingBugstopper @ashleymichal |
the following configurations should be possible. when i say upstream i mean cloudflare's edge OR the --host specified for gcs dev, and when i say local i mean the protocol on the developers machine upstream https + local http (default) upstream http + local https should not be possible i think |
ae738cb
to
4a4b839
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
super close! just that one thing - i have a meeting coming up after that ill check it out and do some QA and then im ready to merge this up.
087882b
to
e453b8f
Compare
e453b8f
to
7100c06
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
fantastic work on this
This will make the
wrangler dev
use https for the local server on gcs and edge. Edge can take in--http
option to use http instead. Borrows some stuff from #1445Closes #1204