-
Notifications
You must be signed in to change notification settings - Fork 3
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
feature: enhance CORS
support
#48
feature: enhance CORS
support
#48
Conversation
a6ac9db
to
65bc7c7
Compare
Hey @rmikalkenas 👋🏻 Thanks for the PR 👍🏻 |
If you have any additional questions - feel free to ping me here or in the our Discord server 👍🏻 |
a2e57a5
to
68794e8
Compare
@rustatian thanks for the comments 👍 updated the PR. Let me know what you think |
68794e8
to
b8c2bb6
Compare
Signed-off-by: Rokas Mikalkėnas <[email protected]>
b8c2bb6
to
77fdb0d
Compare
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.
LGTM, thank you @rmikalkenas 👍🏻
Reason for This PR
roadrunner-server/roadrunner#909
Description of Changes
P.S. will update docs that
allowed_origin
now accepts multiple values separated by comma (if this PR gets approval)License Acceptance
By submitting this pull request, I confirm that my contribution is made under
the terms of the MIT license.
PR Checklist
[Author TODO: Meet these criteria.]
[Reviewer TODO: Verify that these criteria are met. Request changes if not]
git commit -s
).CHANGELOG.md
.