Skip to content

fix: clone authz declarations safely - #36

Merged
intel352 merged 1 commit into
mainfrom
fix/protobuf-clone-vet
May 27, 2026
Merged

intel352 merged 1 commit into
mainfrom
fix/protobuf-clone-vet

Conversation

@intel352

Copy link
Copy Markdown
Contributor

Summary

  • replace value-copy clones for generated authz declaration protobuf messages with proto.Clone
  • deep-clone nested capability requirements to avoid shared mutable protobuf state

Verification

  • GOWORK=off go vet ./...
  • GOWORK=off go test ./...
  • wfctl plugin validate-contract .

@intel352
intel352 merged commit 9e89959 into main May 27, 2026
9 checks passed
@intel352
intel352 deleted the fix/protobuf-clone-vet branch May 27, 2026 11:12
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