-
Notifications
You must be signed in to change notification settings - Fork 5
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
Fix CORS development environment issue #175
Labels
Comments
Adding some technical info that's been gathered for reference: There are several known methods for addressing the issue (including the CORS header mentioned above), three of which are listed below:
There are various pros and cons to diff approaches discussed in this Slack thread. |
amnak613
added a commit
that referenced
this issue
Feb 2, 2025
* updated cors middleware * resolved cors error
This is partially fixed by #184, but there's some configuration that appears to be missing (from eg, |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
@amnak613 identified that we need to account for browser same-origin policy when making client-side API calls (http://localhost:3000 to http://localhost:8000) while working on her branch. This issue is a barrier to testing client-side API calls locally.
The issue is due to not having the origins not being the same and is often solved with a CORS header that gives permission to http://localhost:3000.
Note
NOTE: the plan is for preview and production deployments to use the same origin (host and port), so this issue doesn't currently apply in that scenario. It only currently applies in the dev environment scenario. However, it's possible that the aforementioned deployments could use different origins in the future.
EDIT: Added some technical info for reference in comments: #175 (comment)
The text was updated successfully, but these errors were encountered: