Conversation
…/notary-alpha15 `tlsn` pointed at upstream v0.1.0-alpha.15 while the notary and the keeper pin libid-org/tlsn. The fork's `integration/notary-alpha15` now carries each RCOT consumer on its own domain-separated KOS instance (tlsnotary/tlsn#1173), the verifier sizing the transcript from what it recorded (libid-org/tlsn#2, which drops the declared lengths from `ProveRequestMsg` on the wire), and the mpz fork's executor lifetime fix. A prover on alpha.15 no longer speaks to a verifier on the fork, so this crate pins the fork's head, 94aaaf3, and restates the fork's mpz patch: a dependency's `[patch]` never reaches the workspace that builds it. No source changed: the workspace compiles, clippy is clean, and all 121 tests pass, the in-process MPC-TLS ceremony among them. Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com> Signed-off-by: xgreenx <xgreenx9999@gmail.com>
Contributor
Author
|
Not needed: the root workspaces that build a prover or verifier (notary, keeper) patch |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
tlsnpointed at upstreamv0.1.0-alpha.15; the notary and the keeper pin libid-org/tlsn instead. This moveslibid-tlsnonto the fork at the head ofintegration/notary-alpha15(94aaaf3) and restates the fork's mpz[patch]block (libid-org/mpz1dd2349), because a dependency's own[patch]never applies to the workspace that builds it.What the fork head carries over alpha.15, and why the prover must move with the notary:
0fe3c32(ten commits past alpha.15: #1166, #1167, #1168, #1172);ProveRequestMsg, so a prover on alpha.15 and a verifier on the fork no longer parse each other's messages;94aaaf3).Note: the fork's
mainis still bare upstream at0fe3c32; the libID commits are only onintegration/notary-alpha15.Verification: no source changed;
cargo check --workspace --all-targets, clippy with-D warnings, fmt, andcargo test --workspace(121 passed, including the in-process MPC-TLS ceremony test) all pass locally. MSRV stays 1.95: the mpz fork declares 1.95.Consumers that pin the fork at
816cebc(notary #8's branch, keeper's) should move to94aaaf3and add the same mpz patch, or takelibid-tlsnfrom this commit; libid-server-rs (libid-rs v0.2.0 + upstream alpha.15) cannot talk to a notary on the fork until it does the same.🤖 Generated with Claude Code