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

additional checks for cors #53

Open
chicco785 opened this issue Mar 19, 2022 · 0 comments
Open

additional checks for cors #53

chicco785 opened this issue Mar 19, 2022 · 0 comments
Assignees
Labels
Milestone

Comments

@chicco785
Copy link
Contributor

allow_credentials - Indicate that cookies should be supported for cross-origin requests. Defaults to False. Also, allow_origins cannot be set to ['*'] for credentials to be allowed, origins must be specified.

https://fastapi.tiangolo.com/tutorial/cors/#use-corsmiddleware

Originally posted by @chicco785 in #50 (comment)

also, we have to understand impact on cors of the usage of envoy.

@chicco785 chicco785 added the ci label Mar 31, 2022
@chicco785 chicco785 added this to the 0.4 milestone Mar 31, 2022
@chicco785 chicco785 modified the milestones: 0.4, 0.6, 0.8 Jun 14, 2022
@chicco785 chicco785 modified the milestones: 0.8, 1.0 Oct 21, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants