Skip to content
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

Allow specifying API services in API ML using service ID instead of base path #1449

Open
plavjanik opened this issue Jun 14, 2022 · 1 comment
Labels
enhancement New feature or request priority-low Legit issue but cosmetic or nice-to-have

Comments

@plavjanik
Copy link

Most of the Zowe CLI plug-ins have the parameter base_path that can be used to connect to APIs registered in Zowe API ML.

In such a case you specify the API gateway's hostname and port in the host and port parameters, and basePath is used to specify the path of the URL after port and before the endpoint path.

If I take an example from https://docs.zowe.org/stable/extend/extend-apiml/api-mediation-components-of-url/:

https://ca3x.ca.com:10310/cafilemasterplus/api/v1/mvs/dataSets/test/ping
\______/\_______________/\______________________/\_____________________/
scheme       host             basePath                  endpointPath
                          \______________/\__/\/
                            serviceId      t  v

\_______________________________________________/
                 baseUrl

basePath in this example is /cafilemasterplus/api/v1.

As a user, you need to know what part of the URL is exactly the base path and typical users are not sure if it should be https://ca3x.ca.com:10310/, https://ca3x.ca.com:10310/cafilemasterplus, cafilemasterplus/api/, or /cafilemasterplus/api/v1/.

The worse problem is that the version of the API needs to be specified by the end-user rather than by the CLI plug-in that is developed to work well specific version or set of versions. If a CLI plug-in will need to user api/v2 or api/v3 in the future, it will have to be edited by the end-user.

A possible solution is to accept serviceId as a recommended alternative for basePath. It will be easier for users to find out the correct value of serviceId since it is a single value and is displayed explicitly in the API Catalog. The CLI plug-in will construct the correct URL with the version that is needed for its functionality automatically.

@ATorrise ATorrise added the enhancement New feature or request label Sep 1, 2022
@zFernand0 zFernand0 added enhancement New feature or request and removed enhancement New feature or request labels Mar 8, 2023
@github-actions
Copy link

github-actions bot commented Mar 8, 2023

Thank you for raising this enhancement request.
The community has 90 days to vote on it.
If the enhancement receives at least 5 upvotes, it is added to our development backlog.
If it receives fewer votes, the issue is closed.

@zFernand0 zFernand0 added the priority-low Legit issue but cosmetic or nice-to-have label Mar 8, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request priority-low Legit issue but cosmetic or nice-to-have
Projects
Status: Low Priority
Development

No branches or pull requests

3 participants