Skip to content

Use multi-method audit log search exclusions#200

Draft
yummybomb wants to merge 1 commit into
mainfrom
hypeship/audit-search-sdk
Draft

Use multi-method audit log search exclusions#200
yummybomb wants to merge 1 commit into
mainfrom
hypeship/audit-search-sdk

Conversation

@yummybomb

Copy link
Copy Markdown
Contributor

Summary

  • Updates the Go SDK dependency to the upcoming v0.77 release with list-typed audit-log exclusions.
  • Sends the default GET exclusion and an explicit --exclude-method together through the API.
  • Removes client-side exclusion filtering and its pagination workaround.

Dependency

Blocked on kernel/kernel#2706 and publication of kernel-go-sdk v0.77. The version is intentionally unavailable while this PR is in draft, so go.sum and standard CI dependency resolution will be finalized after publication.

Test plan

  • go test ./... against the generated preview contract
  • go test -race ./cmd -run TestAuditLogsSearch against the generated preview contract
  • go vet ./... against the generated preview contract
  • Verify multiple exclusions serialize as exclude_method=GET,post
  • Run standard CI after kernel-go-sdk v0.77 is published

@kernel-internal

Copy link
Copy Markdown
Contributor

🔧 CI Fix Available

I've pushed a fix for the CI failure. The build failed with missing go.sum entry for github.com/kernel/kernel-go-sdk because go.mod pinned v0.77.0, which isn't published yet (the latest release is v0.76.0). The fix pins the SDK to the published v0.76.0 and joins the exclude methods into the single exclude_method value it accepts, which serializes identically to exclude_method=GET,post — so the multi-method exclusion behavior is unchanged.

👉 Click here to create a PR with the fix

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