-
Notifications
You must be signed in to change notification settings - Fork 61
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
[BUG] Trying APIs on opensearch-project.github.io/opensearch-api-specification is broken #216
Comments
Hey @dblock following this Dockerfile I was able to install the plugin
|
I understood now, the swagger endpoint is coming from https://github.com/opensearch-project/opensearch-api-specification/blob/main/index.html and served with |
@prudhvigodithi That is correct. This issue has nothing to do with the |
Thanks for confirming @dblock, how about we integrate Swagger into an Another approach would be updating the Adding @bbarani @Flyingliuhub |
This is a really good idea. I opened dblock/opensearch-api#3. However, to fix this particular issue I'd prefer if we made |
Yes dB that should work as well, only thing we need to make sure is the API's part of opensearch-openapi.yaml should be same and supported by https://playground.opensearch.org/ (to avoid issues with API change with versioning). To achieve this we just need to update the CORS settings of the cluster and update the Option 1:Add the following settings to playground cluster
Update the opensearch-openapi.yaml with request URL server (example as follows)
Now with Option 2:But if the plugin
|
I like option 1. The spec is compatible with OpenSearch 2.x. Option 2 requires changes to the plugin, the plugin to be installed, and so on, which seems like a lot of work for something we can achieve with option 1. |
@prudhvigodithi Are you going to try and implement this? |
Once we have the CORS enabled and a secure credential for authentication from https://playground.opensearch.org/ we can then come back to modify the opensearch-api-specification swagger setup to talk to playground. @dblock I can contribute to opensearch-api-specification swagger setup but need some changes from https://playground.opensearch.org/ cluster, adding @Flyingliuhub to take a look. |
@prudhvigodithi @Flyingliuhub Is this something that we can do? Is there a GitHub issue for CORS/auth? |
@prudhvigodithi , let's discuss and see what we can do here. |
Sure @Flyingliuhub. |
@prudhvigodithi @Flyingliuhub Any way we can move this forward? |
What is the bug?
Try invoking an API in https://opensearch-project.github.io/opensearch-api-specification/. It fails because it thinks OpenSearch is running on https://opensearch-project.github.io.
What is the expected behavior?
Connect https://opensearch-project.github.io/opensearch-api-specification/ to https://playground.opensearch.org so that APIs can be attempted.
Do you have any screenshots?
The text was updated successfully, but these errors were encountered: