-
Notifications
You must be signed in to change notification settings - Fork 189
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
API redirects & CORS #389
API redirects & CORS #389
Conversation
I tested a few original URLs off this deploy preview and I think we're looking good. @webmasterkai - wanna check? https://deploy-preview-389--upbeat-lovelace-3e9fff.netlify.com/ |
Redirects without query args are preventing proxy
So we still want |
If we set a redirect as |
Got it, thanks! |
If there is a list of used query parameters, I'm fine with putting those in redirects in the interest in getting the bulk of traffic off to static files. Any "new" parameters folks use, of course, won't work, but that means they read the docs and see that things have changed.
|
If nothing else, what we can do here is use Netlify functions to just take those requests, read the static JSON, filter, and respond with the data. |
Added API redirects to the new "V1" prefixed API.
Added CORS header
Fixes #386
Fixes #384