-
Notifications
You must be signed in to change notification settings - Fork 6
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
Adding CORS header on preflight request #80
Conversation
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.
Looks good
Hi!
I'm also starting to wonder if we where testing it the wrong way. Looking at the browser requests it should be more like this
This already works on the main branch, but I had to add these two lines
.. to remove this error I saw in Chrome when testing agains our deployed version
|
@runejo good catch! The middleware worked fine in my test project but it did not i the PxWebApi 2.0 project instead it throw an exception. I have written a fix for it and now the curl query above should work. |
Yes now it works 👍
|
Added a middleware to handle options request