-
Notifications
You must be signed in to change notification settings - Fork 94
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
Update CORS configurations #2101
Update CORS configurations #2101
Conversation
$config[GlueApplicationConstants::GLUE_APPLICATION_DOMAIN] = 'https://glue.mysprykershop.com'; | ||
$config[GlueApplicationConstants::GLUE_APPLICATION_CORS_ALLOW_ORIGIN] = 'https://glue.mysprykershop.com'; | ||
$config[GlueApplicationConstants::GLUE_APPLICATION_REST_DEBUG] = false; | ||
Add the necessary parameters to `deploy.yml`: |
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.
What are the necessary param?
@@ -78,7 +99,22 @@ To make sure that the CORS headers are set up correctly, send the OPTIONS reques | |||
|
|||
* Verify that the `access-control-allow-origin` header is present and is the same to the one set in `config`. |
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.
What's the config
in this context?
@@ -78,7 +99,22 @@ To make sure that the CORS headers are set up correctly, send the OPTIONS reques | |||
|
|||
* Verify that the `access-control-allow-origin` header is present and is the same to the one set in `config`. | |||
* Verify that the `access-control-allow-methods` header is present and contains all available methods. |
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.
How do we know which methods are available?
…' of https://github.com/spryker/spryker-docs into feature/sc-4807/define-cors-allow-origin-via-deploy-yml
PR Description
Ticket - https://spryker.atlassian.net/browse/SC-4807
Checklist