Skip to content

feat(stack-kms-aws): AWS KMS key vendor backend (CIP-5307) - #1626

Draft
freshtonic wants to merge 1 commit into
james/cip-5306-keyset-records-and-derived-data-keysfrom
james/cip-5307-stack-kms-aws
Draft

freshtonic wants to merge 1 commit into
james/cip-5306-keyset-records-and-derived-data-keysfrom
james/cip-5307-stack-kms-aws

Conversation

@freshtonic

Copy link
Copy Markdown
Contributor

Layer 4 of the CIP-5203 stack. Implements CIP-5307 (ADR-0014).

  • New crate stack-kms-aws: a hand-written AWS KMS JSON client with in-crate SigV4 (no aws-sdk). It wraps the root key with Encrypt and unwraps it with Decrypt. The AAD is bound through EncryptionContext.
  • Credentials: static and environment, IRSA, the ECS container endpoint, and IMDSv2 (native only), cached and refreshed before expiry.
  • HttpVendorTransport (native, in stack-kms-vendor).
  • Tests: SigV4 vectors, recorded fixtures, the contract suite, and a LocalStack job that runs on PRs.
  • The crate is staged for crates.io, with release = false.

Decisions are in docs/plans/2026-11-05-key-vendor-backends-decisions.md (CIP-5307 sections).

@coderabbitai

coderabbitai Bot commented Nov 12, 2026

Copy link
Copy Markdown
Contributor

Important

Draft PR not reviewed

Draft PRs are not automatically reviewed by default.

  • Trigger a manual review

To automatically review draft PRs, update your CodeRabbit configuration:

reviews:
  auto_review:
    drafts: true

@changeset-bot

changeset-bot Bot commented Nov 12, 2026

Copy link
Copy Markdown

⚠️ No Changeset found

Latest commit: f07f74a

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

@github-actions

Copy link
Copy Markdown
Contributor

Mutation testing (cargo-mutants, --in-diff, stack-auth + stack-encrypt)

cargo-mutants did not complete (exit ) before producing results. Check the workflow run logs. The gate below will fail.

This branch has not been deployed

No deployments
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