Skip to content

Tighten the semantics of POST /v3/service_plans/:guid/visibility#4609

Merged
philippthun merged 1 commit intocloudfoundry:mainfrom
sap-contributions:post-visibilities-fix
Mar 26, 2026
Merged

Tighten the semantics of POST /v3/service_plans/:guid/visibility#4609
philippthun merged 1 commit intocloudfoundry:mainfrom
sap-contributions:post-visibilities-fix

Conversation

@philippthun
Copy link
Member

  • Allow POST only when type = organization. Otherwise fail with 422 Unprocessable Entity.
  • Ensure the target plan already has visibility type organization. If the plan’s visibility type is not organization, the request fails with 422 Unprocessable Entity.

Fixes #4606

  • I have reviewed the contributing guide

  • I have viewed, signed, and submitted the Contributor License Agreement

  • I have made this pull request to the main branch

  • I have run all the unit tests using bundle exec rake

  • I have run CF Acceptance Tests

- Allow POST only when type = organization. Otherwise fail with 422
  Unprocessable Entity.
- Ensure the target plan already has visibility type organization. If
  the plan’s visibility type is not organization, the request fails with
  422 Unprocessable Entity.
@philippthun philippthun force-pushed the post-visibilities-fix branch from 7dd0212 to 8bcb21f Compare March 24, 2026 15:01
@philippthun philippthun merged commit 1306e1c into cloudfoundry:main Mar 26, 2026
17 of 18 checks passed
ari-wg-gitbot added a commit to cloudfoundry/capi-release that referenced this pull request Mar 26, 2026
Changes in cloud_controller_ng:

- Tighten the semantics of POST /v3/service_plans/:guid/visibility
    PR: cloudfoundry/cloud_controller_ng#4609
    Author: Philipp Thun <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Restrict POST /v3/service_plans/:guid/visibility to only support appending orgs (and not changing visibility type)

2 participants