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
OAUTH2-207 Add Feature to register CORS Filter #691
Conversation
…xecute after AuthVerifierFilter
To conserve resources, the PR Tester does not automatically run for every pull. If your code changes were already tested in another pull, reference that pull in this pull so the test results can be analyzed. If your pull was never tested, comment "ci:test" to run the PR Tester for this pull. |
...-provider-rest/src/main/java/com/liferay/oauth2/provider/rest/internal/cors/CORSFeature.java
Show resolved
Hide resolved
ci:test:sf |
❌ ci:test:sf - 0 out of 1 jobs passed in 1 minute 31 seconds 204 msClick here for more details.Base Branch:Branch Name: master Sender Branch:Branch Name: OAUTH2-207 1 Failed Jobs:For more details click here.Please fix rebase errors on martamedio/OAUTH2-207
|
Closing CORS |
We have added a Feature to register CORS Filter as we discussed.
cc/ @csierra
Edited: We have also studied the possibility of not implementing a Feature and using a ContainerRequestFilter/ContainterResponseFilter, but involves some changes to avoid code duplication (for parsing headers and get the OAuth2 application).
So far I have left it with the Feature format, we look forward to your comments.