Skip to content
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

Remove tiangolo.com from the CORS origin list #3

Closed
wants to merge 1 commit into from
Closed

Remove tiangolo.com from the CORS origin list #3

wants to merge 1 commit into from

Conversation

ferndot
Copy link

@ferndot ferndot commented Mar 19, 2019

This should not be enabled on every project unless tiangolo.com provides functionality necessary for the project to work.

This should not be enabled on every project unless tiangolo.com provides functionality necessary for the project to work.
@tiangolo
Copy link
Member

Yes, it provides functionatily for development.

http://localhost.tiangolo.com points to your localhost (127.0.0.1)
http://local.dockertoolbox.tiangolo.com points to the default Docker Toolbox IP if you have Docker Toolbox installed (192.168.99.100)

Both are local IPs.

Having a domain pointing to them allows debugging certain things that are not easy to debug otherwise (like CORS configurations).

@ferndot
Copy link
Author

ferndot commented Mar 20, 2019

Thanks for clarifying! I should have looked deeper in the docs.

@ferndot ferndot closed this Mar 20, 2019
@ferndot ferndot deleted the patch-1 branch March 20, 2019 19:17
@tiangolo
Copy link
Member

No problem, thanks for coming back and closing it 👍

@tabuchid
Copy link

tabuchid commented Aug 2, 2021

@tiangolo Just out of curiosity. How have you setup those domains to point at localhost? I recently was trying to do something similar using route53 and couldn't get it to work.

alejsdev pushed a commit that referenced this pull request Dec 19, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants