fix(ngts): refresh NGTS spec (path cleanup + fingerprint field clarification) - #1468
Merged
rajitshah-ca merged 2 commits intoSep 15, 2026
Conversation
Regenerates the TLS Protect Cloud OpenAPI spec from the latest ua-repo pan.dev pipeline run to pick up two changes already live upstream: - Certificate endpoint paths dropped the /outagedetection prefix (e.g. /outagedetection/v1/certificates -> /v1/certificates), matching the same change already deployed to developer.venafi.com by the same pipeline run. - "Built-in Accounts" tag casing corrected to match ua-repo's source of truth (siteconf-pandev.yaml), which was fixed after the original capitalized version was published here. EPIC: VC-50859 Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
charissaf
approved these changes
Sep 10, 2026
|
Visit the preview URL for this PR (updated for commit a9a6c89): https://pan-dev-f1b58--pr1468-sztqzd6d.web.app (expires Tue, 22 Sep 2026 07:00:05 GMT) 🔥 via Firebase Hosting GitHub Action 🌎 Sign: 9359a9fa100d5fdb082f75280473a90b1f96eb51 |
Regenerates the NGTS spec from ua-repo pipeline 2848060423 to pick up one content change: CertificateRequestRequest.existingCertificateFingerprint's description now specifies it's a SHA-1 fingerprint. EPIC: VC-50859 Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
This branch was successfully deployed
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
What
Refreshes the NGTS (Venafi TLS Protect Cloud) OpenAPI spec on pan.dev from the latest ua-repo pan.dev pipeline runs:
Changes
/outagedetectionprefix (e.g./outagedetection/v1/certificates->/v1/certificates). This matches the same path change already deployed todeveloper.venafi.comby the same pipeline run's Dev Central job (Deploy API Production), so this brings pan.dev in line with what's already live on the primary docs.Built-In AccountstoBuilt-in Accounts, matching ua-repo's source of truth (siteconf-pandev.yaml), which was corrected there shortly after the capitalized version was originally published to this repo and never republished.existingCertificateFingerprintfield description clarified:CertificateRequestRequest.existingCertificateFingerprintnow specifies it's a SHA-1 fingerprint (was previously just "Fingerprint of an existing inventory certificate...").containerFormatquery param onGET /v1/certificates/{id}/contents, and a newPOST /v1/machines/workflowsendpoint (machines_initiateTestConnectionWorkflow), carried over from the regenerated spec.No Docusaurus config, sidebar, or generated page changes needed — generated files aren't committed (built at deploy time), consistent with #1311.
Motivation and Context
EPIC: VC-50859
How Has This Been Tested
siteconf-pandev.yaml,operationsDoc/) and confirmed the path change is already live on developer.venafi.com via the same pipeline run.existingCertificateFingerprintwording change and rule out unrelated component$refrenumbering noise.🤖 Generated with Claude Code