Skip to content

PROP-220: verify chain-bound Repl creation timestamps - #50

Open
pellicceama wants to merge 1 commit into
mainfrom
prop-220-repl-created-at-identity
Open

pellicceama wants to merge 1 commit into
mainfrom
prop-220-repl-created-at-identity

Conversation

@pellicceama

Copy link
Copy Markdown

What changed

The Go Repl identity verifier now exposes a Repl creation time only when a certificate claim binds the value. It rejects a body that differs from the claim and ignores an unbound body value, so workload code cannot backdate a Repl with REPL_IDENTITY_KEY.

This PR mirrors the contract from https://github.com/replit/goval/pull/22816. It also preserves the timestamp when a caller signs an audience-specific child identity.

References PROP-220

Test plan

  • go test ./...

Rollout

Release this module after the Goval contract merges. Goval must update its Vault renewal consumer to this release before web starts sending the timestamp.

This change is additive and safe to revert before timestamp emission starts.

~ written by Zerg 👾 (volatile-tempest-e716)

@pellicceama pellicceama added the zergling-authored PR authored by a Zerg agent label Aug 27, 2026
@codecov

codecov Bot commented Aug 27, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 78.72340% with 10 lines in your changes missing coverage. Please review.
✅ Project coverage is 52.42%. Comparing base (2b01855) to head (9ea4721).

Files with missing lines Patch % Lines
verify.go 56.52% 6 Missing and 4 partials ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main      #50      +/-   ##
==========================================
+ Coverage   51.21%   52.42%   +1.20%     
==========================================
  Files           7        7              
  Lines         697      723      +26     
==========================================
+ Hits          357      379      +22     
- Misses        291      293       +2     
- Partials       49       51       +2     

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@yinganwang
yinganwang marked this pull request as ready for review September 1, 2026 03:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

zergling-authored PR authored by a Zerg agent

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants