Repository navigation
feat!: align SDK with merged ANS specs and Verified Identities - #127
Draft
csnitker-godaddy wants to merge 1 commit into
Draft
csnitker-godaddy wants to merge 1 commit into
csnitker-godaddy wants to merge 1 commit into
Conversation
Add the V2 management and identity surface and close the shared naming, DNSSEC, badge-version, and SCITT verification gaps. Preserve V1 reads and Method B's explicit receipt waiver; keep V3 out of scope. Fixes agentnameservice#82 Fixes agentnameservice#118 BREAKING CHANGE: identity_csr_pem is optional, invalid host/version inputs are rejected, and outgoing SCITT suppliers require explicit initialization.
This branch has not been deployed
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.
Summary
The SDK lacks merged V2 management and Verified Identity features, and several verification paths accept evidence that does not satisfy the merged ANS trust requirements. This change adds the missing client surface and closes those verification gaps against ans-registry f2373f1.
V1 reading and Method B's explicit receipt waiver remain supported. V3 is excluded; LEI/ENS proof profiles remain postponed by the merged spec. Historical proof verification does not establish current status or audit completeness. See
docs/SPEC-COMPLIANCE.mdfor the contract matrix and trust boundaries.Compatibility
identity_csr_pembecomesOption<String>; existing registration constructors still preserve supplied values.Fqdn/version parsing rejects invalid forms. CallScittHeaderSupplier::initializebefore serving requests. Migration examples and behavior are documented inMIGRATING.md.Stack
Depends on #126, based exactly on
3a45dbccbf0f40941b02b7f12cd205ae67b2bd3d. The base branch mirrors that dependency head so this diff contains only the feature/compliance work. Merge #126 first, then retarget this PR tomainbefore merging.Validation
All 12 GitHub Actions checks pass on
998b044a417425eb66c24b5ba6440c890b4be1f7. DCO remains pending human certification.Fixes #82
Fixes #118
AI assistance
Assisted-by: OpenAI Codex (GPT-6)
Draft pending the human submitter's DCO certification. No Signed-off-by trailer has been added for this feature commit.