[VPI-331] - Use DNS suffix in token exchange guide examples - #117
Conversation
Co-authored-by: Cursor <cursoragent@cursor.com>
|
There was a problem hiding this comment.
Cursor Bugbot has reviewed your changes using high effort and found 1 potential issue.
❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.
Reviewed by Cursor Bugbot for commit 553b38a. Configure here.
|
|
||
| The examples in this guide use `platform.[DNS_SUFFIX]`, where `[DNS_SUFFIX]` is | ||
| your deployment's DNS suffix. For example, if your DNS suffix is | ||
| `dev.example.com`, the Platform API is at `https://platform.dev.example.com`. |
There was a problem hiding this comment.
Missing private-cloud-only callout
Medium Severity
The platform.[DNS_SUFFIX] examples treat the deployment DNS suffix as a general setting without a private-cloud-only callout or the public cloud host vertexvis.com. Public Vertex cloud customers cannot configure a DNS suffix and may not know which API host to use.
Triggered by learned rule: Call out private-cloud-only features
Reviewed by Cursor Bugbot for commit 553b38a. Configure here.





Agent:(claude-opus-5.5)Summary
Follow-up to #116 for the token exchange OIDC configuration guide:
http://endpoints withALLOW_HTTP_OIDC_ENDPOINTS.That's an internal development setting, not something customers configure.
platform.vertexvis.comtoplatform.[DNS_SUFFIX], with a short explanation, so they also work forprivate cloud deployments.
Test Plan
docusaurus buildand Prettier pass.Release Notes
None.
Possible Regressions
None. Docs only.
Dependencies
None.
Related
VPI-331 · #116
Note
Low Risk
Documentation-only edits with no runtime or API behavior changes.
Overview
Updates the token exchange OIDC configuration guide so examples work for private cloud and other non–
vertexvis.comdeployments.Endpoint requirements: Drops the customer-facing bullet about requiring
https://and the internalALLOW_HTTP_OIDC_ENDPOINTSoverride.API examples: Replaces hard-coded
https://platform.vertexvis.comURLs withhttps://platform.[DNS_SUFFIX]and adds a short note explaining how to substitute your deployment’s DNS suffix (e.g.dev.example.com→https://platform.dev.example.com) for token, account create/update, and account GET curls.Reviewed by Cursor Bugbot for commit 553b38a. Bugbot is set up for automated code reviews on this repo. Configure here.