Skip to content

Return last consistency proof for distinguished queries - #19

Open
Ap4sh wants to merge 1 commit into
signalapp:mainfrom
Ap4sh:fix-distinguished-last-consistency
Open

Return last consistency proof for distinguished queries#19
Ap4sh wants to merge 1 commit into
signalapp:mainfrom
Ap4sh:fix-distinguished-last-consistency

Conversation

@Ap4sh

@Ap4sh Ap4sh commented Jun 5, 2026

Copy link
Copy Markdown

fixes #11

summary

  • map the distinguished request's last tree size into both returned consistency proof slots
  • keep the existing distinguished proof behavior while also returning the missing Last proof
  • keep kt-client stateless verification working when distinguished lookups use -last
  • add regression coverage for the server proof response and the client-side consistency proof cleanup

testing

  • go test ./cmd/kt-client -run TestRemoveConsistencyProofsForStatelessVerification -count=1
  • go test ./cmd/kt-client -count=1
  • go test ./cmd/kt-server -run TestDistinguished_LastReturnsConsistencyProof -count=1
  • go build ./...
  • go test -count=1 ./...
  • (cd filter-key-updates && ./mvnw verify)

@Ap4sh
Ap4sh marked this pull request as ready for review June 6, 2026 00:02
@Ap4sh
Ap4sh force-pushed the fix-distinguished-last-consistency branch from b369e29 to 1186b0c Compare June 6, 2026 13:40
@Ap4sh
Ap4sh force-pushed the fix-distinguished-last-consistency branch from 1186b0c to 4ed8099 Compare July 5, 2026 22: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.

No Consistency Proofs are provided as distinguished.Last is ignored

1 participant