Skip to content

add permissions to allow OPTIONS request #2

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

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

esteban-serfe
Copy link

Edited pound.cfg config file to allow to pass OPTIONS request into the Magento instance.

This merge allows to pass from TSL container all the request using the OPTIONS header into the chain of request. This is required when using CORS on the REST API calls.

Description

To allow to make the OPTIONS calls from the pound system into the chain up to Magento to decide the response, we need to allow pound to handle this http method.
Without this change the OPTIONS call returns a 501 Not implemented error code on the OPTION call.
This is only required when using CORS to allow to use the Magento API from another domain.

Fixed Issues (if relevant)

Cannot send issues into the repository as the option is not available

Manual testing scenarios

Without the merge request:
curl -v --location --request OPTIONS 'https://magento2.docker/rest/all/V1/integration/admin/token''
Response code is 501 Not implemented

Contribution checklist

  • Pull request has a meaningful description of its purpose
  • All commits are accompanied by meaningful commit messages

Edited pound.cfg config file to allow to pass OPTIONS request into the Magento instance.
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.

1 participant