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

Parameterizing protocol so we can use https instead of http #1

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

cfe84
Copy link

@cfe84 cfe84 commented Aug 21, 2019

I added an environment variable to override protocol so we can deploy the api with HTTPS and get rid of the mixed content errors

@benc-uk
Copy link
Contributor

benc-uk commented Oct 7, 2019

Another approach would be to use // rather than http:// that way the requests will match the origin either http or https depending on how the site was loaded

https://www.paulirish.com/2010/the-protocol-relative-url/

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.

2 participants