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

Fix CORS OPTIONS Call for POST REST requests #163

Closed
wants to merge 3 commits into from
Closed

Fix CORS OPTIONS Call for POST REST requests #163

wants to merge 3 commits into from

Conversation

SalvatorePollaci
Copy link
Contributor

I have found that all OPTIONS calls for POST REST requests return 405 error message, this is due to the fact that DefaultRouter uses Symfony's default RequestContext which refers to a GET request. Therefore I'm setting the the RequestContext given the Request Object.

@andrerom
Copy link
Contributor

andrerom commented Jan 16, 2019

+1, should this be applied to 1.4 as well? (for eZ Platform 1.x)

@andrerom andrerom requested a review from emodric January 16, 2019 10:35
@andrerom
Copy link
Contributor

@SalvatorePollaci Up for rebasing, or do you prefer we do that for the merge? :)

@SalvatorePollaci
Copy link
Contributor Author

@andrerom honestly I would prefer that you guys rebase. Thanks 👍

@andrerom
Copy link
Contributor

ok

andrerom pushed a commit that referenced this pull request Jan 17, 2019
- Set RequestContext from Request in DefaultRouter
@andrerom
Copy link
Contributor

Fixed in ab86627

@andrerom andrerom closed this Feb 18, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

5 participants