Skip to content

Remove legacy HTTP method wrappers from ApiClient - #216

Open
cx-happy-yang wants to merge 1 commit into
masterfrom
refactor/remove-legacy-http-request-methods
Open

Remove legacy HTTP method wrappers from ApiClient#216
cx-happy-yang wants to merge 1 commit into
masterfrom
refactor/remove-legacy-http-request-methods

Conversation

@cx-happy-yang

Copy link
Copy Markdown
Contributor

Summary

  • Remove head_request, get_request, post_request, put_request, patch_request, and delete_request from ApiClient
  • All six were thin one-liner wrappers around call_api with no remaining callers — the entire SDK (CxOne, CxRestAPISDK, CxScaApiSDK, CxAccessControl, CxReporting, CxODataApiSDK) has been fully migrated to call_api
  • Verified with grep that no code in the SDK, tests, or examples references these methods

Test plan

  • Run the full test suite to confirm no regressions
  • Grep for \.get_request\|\.post_request\|\.put_request\|\.patch_request\|\.delete_request\|\.head_request to confirm zero remaining usages

🤖 Generated with Claude Code

head_request, get_request, post_request, put_request, patch_request,
and delete_request were thin wrappers around call_api with no remaining
callers — all SDK modules have been migrated to call_api directly.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@github-actions

Copy link
Copy Markdown

Logo
Checkmarx One – Scan Summary & Details56db01bb-073d-4254-9a69-6efc518a794b

Great job! No new security vulnerabilities introduced in this pull request

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.

1 participant