Skip to content

SYN-5402: client certificates - #42

Merged
borisvalo merged 5 commits into
v2from
syn-5402-client-certificates
Jun 24, 2026
Merged

SYN-5402: client certificates#42
borisvalo merged 5 commits into
v2from
syn-5402-client-certificates

Conversation

@borisvalo

Copy link
Copy Markdown
Contributor

Resolves #SYN-5402


Before the change?

  • syntheticsclient/v2 does not expose Synthetics client certificate CRUD APIs.
  • HTTP V2, Browser V2, and API V2 client models do not expose client certificate attachment fields.
  • Public request details can include raw API tokens, certificate content, passwords, header values, cookie values, or authentication values.

After the change?

  • Adds public ClientCertificate* models using public API JSON casing.
  • Adds create, get, list, update, and delete methods for /certificates.
  • Adds HTTP certificateId, Browser advancedSettings.certificateIds, and API request configuration.certificateId model support.
  • Redacts API token, content, password, and value fields from captured request details.
  • Removes raw request dump printing from the public API client path.

Pull request checklist

  • Acceptance Tests have been updated, run (make testacc), and pasted in this PR (for bug fixes / features)
  • Docs have been reviewed and added / updated if needed (for bug fixes / features)

Acceptance Test Output

PASS
coverage: 71.8% of statements
ok github.com/splunk/syntheticsclient/v2/syntheticsclientv2

Does this introduce a breaking change?

  • Yes
  • No

@borisvalo borisvalo self-assigned this Jun 22, 2026
@borisvalo
borisvalo force-pushed the syn-5402-client-certificates branch from c98f475 to 63ada57 Compare June 24, 2026 08:45
@borisvalo
borisvalo merged commit c0c5308 into v2 Jun 24, 2026
2 of 4 checks passed
@github-actions github-actions Bot locked and limited conversation to collaborators Jun 24, 2026
@jcsco
jcsco deleted the syn-5402-client-certificates branch July 31, 2026 20:58
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants