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

Undocumented API #582

Closed
nebhale opened this issue Apr 25, 2016 · 6 comments
Closed

Undocumented API #582

nebhale opened this issue Apr 25, 2016 · 6 comments

Comments

@nebhale
Copy link
Member

nebhale commented Apr 25, 2016

Issue

The CF CLI makes a request for GET /v2/service_instances/<SERVICE_INSTANCE_ID>/service_keys which is an undocumented API.

Context

While tracing the cf service-keys command against PWS (2.53.0), I noticed the following request sequence:

GET /v2/spaces/<SPACE_ID>/service_instances?return_user_provided_service_instances=true&q=name%3A<SERVICE-INSTANCE-NAME>&inline-relations-depth=1
GET /v2/services/<SERVICE_ID>
GET /v2/service_instances/<SERVICE_INSTANCE_ID>/service_keys

The final request is not documented.

Steps to Reproduce

CF_TRACE=true cf set-keys <SERVICE-INSTANCE-NAME>

@cf-gitbot
Copy link

We have created an issue in Pivotal Tracker to manage this. You can view the current status of your issue at: https://www.pivotaltracker.com/story/show/118333745.

@ameowlia
Copy link
Member

Hi @nebhale,

Thanks, nice catch. We added docs in this commit. The docs should be updated soon.

-- @adowns01 && @simonleung8, CAPI Team Members

@nebhale
Copy link
Member Author

nebhale commented May 4, 2016

@SocalNick I'm not sure this quite complete. Reviewing the release-candidate docs, it does not appear that the name filter parameter is documented.

@SocalNick
Copy link
Contributor

Re-opened tracker issue

@mikexuu
Copy link
Contributor

mikexuu commented Dec 16, 2016

Hey @nebhale,

This seems to be documented now. Thanks for bringing up this issue. We are closing this now, feel free to reopen if you experience any other problems.

Thanks,
CAPI Community Pair - @jberkhahn & @michaelxupiv

@mikexuu mikexuu closed this as completed Dec 16, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

5 participants