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

CORS Support #931

Closed
romain-grecourt opened this issue Aug 20, 2019 · 5 comments · Fixed by #1633
Closed

CORS Support #931

romain-grecourt opened this issue Aug 20, 2019 · 5 comments · Fixed by #1633
Assignees
Labels
Milestone

Comments

@romain-grecourt
Copy link
Contributor

@m0mus m0mus added enhancement New feature or request SE labels Aug 22, 2019
@barchetta barchetta added the P3 label Oct 24, 2019
@spericas spericas self-assigned this Oct 29, 2019
@spericas spericas changed the title WebServer: Implement CORS support CORS and CSRF Support Oct 29, 2019
@spericas spericas changed the title CORS and CSRF Support CORS Support Oct 30, 2019
@spericas
Copy link
Member

CORS support for MP in progress here: https://github.com/spericas/helidon/tree/cors/microprofile/cors

@spericas
Copy link
Member

spericas commented Nov 6, 2019

CORS wiki page is now here:
https://github.com/oracle/helidon/wiki/CORS-in-Helidon-MP

@spericas spericas added this to the 2.0.0 milestone Nov 7, 2019
@rafabene
Copy link

Hi @spericas . Any updates on microprofile-cors? I've created a custom filter for my applications but it seems that /openapi doesn't get intercepted by the filter.

@spericas
Copy link
Member

@rafabene Hoping to get that in the next milestone, at least for MP. Not sure that would help OpenAPI as those resources are a bit different. What do you think @tjquinno?

@rafabene
Copy link

rafabene commented Mar 24, 2020

@spericas Thanks for having this as part of next milestone. For OpenAPI, I provided the following PR #1566

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
Archived in project
Development

Successfully merging a pull request may close this issue.

6 participants