-
Notifications
You must be signed in to change notification settings - Fork 2
feat: allow CORS from dotcom local dev #384
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
base: main
Are you sure you want to change the base?
Conversation
| plug Plug.Head | ||
| plug Plug.Session, @session_options | ||
|
|
||
| plug CORSPlug, origin: ["http://localhost:4001"] |
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.
@boringcactus maybe you should put it in the :api pipeline in the router? For whatever reason it isn't working for me 😬
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.
Putting it here is recommended by the docs, since if we put it in the router we have to also declare OPTIONS handlers in addition to our GET handlers.
Summary
Ticket: none
Requested by the dotcom team for their exploration of the JS export.