-
Notifications
You must be signed in to change notification settings - Fork 75
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
General CORS library #62
Comments
Hey and thanks for your input. I understand the need for a generalized CORS library. I wonder if the plan you laid out is the way to go or whether we should have a separate package (maybe creatively named Not sure which way I prefer... |
I can see why the second one might sound a good idea, but perhaps later on. While the API is being developed/discovered I normally think it is better to have everything in one repo. Main reason I would do as you suggest is because I'm not sure how well hex.pm handles renaming packages |
I've started a spike at a CORS middleware for Raxx. https://github.com/CrowdHailer/raxx/pull/157/files#diff-145629a0af7f990c2e3f756d6625052aR7 I now reckon just exposing a |
This is pretty close to a feature request, but it looks like the right place for it and I'm willing to do much of the work.
Is there any interest in making this a general CORS library, i.e. not just a plug.
I think work needed would be
Code.ensure_loaded?
cors
on hex. It is still available.Why.
a) Useful in tests of APIs to check that the headers include the expected CORS headers
b) I, and quite a few others, use Raxx/Ace
The text was updated successfully, but these errors were encountered: