Skip to content

feat(proto): add Integrity and SignedAccessRecord message types#71

Merged
ghaskins merged 1 commit into
manetu:mainfrom
ghaskins:add-signed-accessrecord-pb
Jun 29, 2026
Merged

feat(proto): add Integrity and SignedAccessRecord message types#71
ghaskins merged 1 commit into
manetu:mainfrom
ghaskins:add-signed-accessrecord-pb

Conversation

@ghaskins

Copy link
Copy Markdown
Contributor

Adds Integrity and SignedAccessRecord envelope messages to the shared events/v1 protobuf schema to support tamper-resistant audit trails. SignedAccessRecord carries verbatim AccessRecord bytes alongside a cryptographic leaf_hash and signature. Note: SignedAccessRecord is not wire-compatible with the existing AccessRecord Kafka topic and requires a separate transport.

Adds Integrity and SignedAccessRecord envelope messages to the shared
events/v1 protobuf schema to support tamper-resistant audit trails.
SignedAccessRecord carries verbatim AccessRecord bytes alongside a
cryptographic leaf_hash and signature. Note: SignedAccessRecord is not
wire-compatible with the existing AccessRecord Kafka topic and requires
a separate transport.

Signed-off-by: Greg Haskins <greg@manetu.com>
@codecov

codecov Bot commented Jun 29, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 8.45070% with 65 lines in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
...protos/manetu/policyengine/events/v1/message.pb.go 8.45% 65 Missing ⚠️

📢 Thoughts on this report? Let us know!

@muralisrini muralisrini left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

@ghaskins ghaskins merged commit 16ce016 into manetu:main Jun 29, 2026
16 of 17 checks passed
@ghaskins ghaskins deleted the add-signed-accessrecord-pb branch June 29, 2026 18:49
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.

2 participants