Skip to content

Hide unavailable loggregator endpoints from / response - #5117

Merged
philippthun merged 1 commit into
cloudfoundry:mainfrom
sap-contributions:fix-issue-3603-null-logging-endpoints
Jul 28, 2026
Merged

Hide unavailable loggregator endpoints from / response#5117
philippthun merged 1 commit into
cloudfoundry:mainfrom
sap-contributions:fix-issue-3603-null-logging-endpoints

Conversation

@philippthun

Copy link
Copy Markdown
Member

Use a nil-when-blank helper for the logging, log_cache, and log_stream links in the root controller, matching the existing pattern used by credhub_link and routing_link. The link entry becomes nil when the underlying URL is not configured, instead of advertising an empty href.

  • 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

Use a nil-when-blank helper for the logging, log_cache, and log_stream
links in the root controller, matching the existing pattern used by
credhub_link and routing_link. The link entry becomes nil when the
underlying URL is not configured, instead of advertising an empty href.
@philippthun
philippthun merged commit 7431ea4 into cloudfoundry:main Jul 28, 2026
11 checks passed
ari-wg-gitbot added a commit to cloudfoundry/capi-release that referenced this pull request Jul 28, 2026
Changes in cloud_controller_ng:

- Bump UAA devcontainer to SapMachine JDK 25
    PR: cloudfoundry/cloud_controller_ng#5309
    Author: Philipp Thun <philipp.thun@sap.com>

- Hide unavailable loggregator endpoints from / response
    PR: cloudfoundry/cloud_controller_ng#5117
    Author: Philipp Thun <philipp.thun@sap.com>

- Document VCAP_SERVICES to service binding files translation
    PR: cloudfoundry/cloud_controller_ng#5314
    Author: Philipp Thun <philipp.thun@sap.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Removing loggregator endpoints from advertised endpoints when unavailable

2 participants