Repository navigation
gdc: pins to the current integration builds, with what went into each (supersedes #79 and #80) - #81
Merged
Merged
Conversation
…image overlay that bakes in the circuit artifact Preparation only: the pinned verifier image predates SUNET/vc#832 and ignores the new keys. Helm renders artifact_bytes (1062842) as 1.062842e+06, which became 1062842.0; normalised to an integer for the allowlist. Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
… re-issuing the signing cert from the same key and root The vc-verifier's client_id is x509_san_dns:<host> and it signs request objects with the shared signing certificate, whose SAN lacked that host. Deploy passes the verifier's public host; signing_cert_current() checks it exactly (no substring or wildcard match). Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
…erifier.signingCertDnsSan), set only for gdc Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
…--chmod made them 0444, so the non-root verifier could not open it) Found by starting the v5 verifier image locally: 'open /etc/vc/circuits/nym_auth-ssc.bin: permission denied'. Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
…rifier branch (PR 79) Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
…t overlay) Unmerged upstream work; PRs 79 and 80 are NOT merged and this branch contains both. Re-pin to a released vc verifier image when SUNET/vc releases the verifier stack. Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
…system_values; verifier_id is the prefixed client_id) Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
…ct overlay); merge the v0.4 config PRs 79 and 80 are NOT merged and this branch contains both; re-pin to a released vc verifier when SUNET/vc releases the verifier stack. Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
…prefix) + artifact overlay Unmerged upstream work (PRs 79 and 80 are not merged); re-pin to a released vc verifier when released. Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
…hape per vc's brief), with a test Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
…094549cc build cannot answer a zkp_req request) + artifact overlay Unmerged upstream work (PRs 79 and 80 are not merged); re-pin to a released vc verifier when released. Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
…0-12-31) Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
…+ #471 + #163 + #472, unmerged, local branch) Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
…63 + #471 narrowed; unmerged, local branch) Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
…leased; not deployed) Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
…ENANCE block naming each pin's tag, digest, base and PR heads All local builds of unmerged PRs; re-pin to released images when they land. Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
This was referenced Oct 21, 2026
Merged
…pin provenance updated Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
…horized offer twins They are issuance-only scopes; listing them doubled the verifier's /ui/metadata and picker (26 -> 14 on gdc). The issuer keeps its twins (offers resolve against credential_configurations_supported); minting is unchanged. Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
…ase.yaml's signingCertDnsSan from #80) Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
…wins chart: the verifier no longer lists the offer twins (step 1; 26 -> 14 on gdc)
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.
gdc runs exactly this branch (2026-10-21), and every pinned image below is a LOCAL BUILD OF UNMERGED PULL REQUESTS. This PR records the pins and what went into each build, so the team can see it. It contains and supersedes #79 (nym_auth verifier config + circuit artifact overlay) and #80 (verifier dNSName in the signing cert, opt-in, same key and root): review and merge this one and close those two, or merge them first and rebase this one. Nothing here is merged upstream; each pin must be re-pinned to a released image when its PRs land.
Pins (also written as a PIN PROVENANCE comment block at the top of
images:inenvironments/gdc.yaml)local-1792603698/sha256:96c41f16dd9324da6890c76b9faa7810d4c8259cf142ffd16360e413c8ca8945integration/gdc-nymauth-v5@a56c4be9ec42ef394d536f5ea08d80b50f2a95a0(merge base SUNET/vc main4bba38e614e1), zknative, + the circuit artifact overlaylocal-1792473614/ apigwsha256:6d532929…eb27, issuersha256:c0efa58f…8b58, registrysha256:458cb70c…3268integration/gdc-nymauth-v4@5e546b2d61d4756f04b39ea84d26cf6b18b830balocal-1792564143/sha256:285a7ca4ee1eaa27cacbe45435965ba08b15e2e98b680636a29ee5073da97749local/gdc-nymauth-integration-3@9cb6e57d8498f0a1efa024254a6e99e2dbf278ab, a LOCAL branch that is never pushed; base origin/maincb42861b6d640ea17166e6491f69761f58a5f172ghcr.io/sirosfoundation/go-trust:0.26.0(released; index digestsha256:73d994eaabcf27d68837c12ef42cbb16072c87562c6abe5cf7e2332a8068cf1e)values.pdp.extraConfig; deployed 2026-10-21What is in each build (PR heads that were built)
Verifier (vc v5): SUNET/vc#827 @ 80d6ee9ef, SUNET/vc#829 @ 55a8c9d7f, SUNET/vc#830 @ 770885f31 (incl. the allow-all-PDP fix), SUNET/vc#831 @ deb0c19b6, SUNET/vc#832 @ 6649047e4 (all open), plus the rulebook v0.4 port, the verifier UI fixes (
fn_idin req-schema.js; postedreqvs wire_req) and themeta.source_formatfix, on that integration branch. ZK libs https://github.com/sirosfoundation/zk-cred-longfellow v0.1.2 @58de20f0, https://github.com/sirosfoundation/zk-cred-vega v0.0.7 @8e7be172; matcher https://github.com/sirosfoundation/siros-dc-matcher v0.11.0; circuit catalog https://github.com/sirosfoundation/go-zk-circuits release v0.1.0-release.16 served from zk-circuits.fly.dev.Issuer side (vc v4): SUNET/vc#793 @ 3bfa96b6e (seed-backed commitments; merged upstream since), SUNET/vc#821 @ fd078a057 (seed delivery; open), SUNET/vc#813 @ 2e69dd2b4, SUNET/vc#796 @ 62c93fa01 and SUNET/vc#820 @ 6db9dd19b (closed; superseded), on SUNET/vc main as of 2026-10-20 (so also upstream #809 and #790).
Engine: signed
--no-ffmerges on origin/main of sirosfoundation/go-wallet-backend#469 @ e4e0bc3b (credential request encryption), sirosfoundation/go-wallet-backend#471 @ f1362658 (client_metadata / verifier trust context, incl. the fail-closed fix of sirosfoundation/go-wallet-backend#473), sirosfoundation/go-wallet-backend#163 @ 71a07f80 (WMP, go-wmp v0.7.0), sirosfoundation/go-wallet-backend#472 @ be69e2e7 (sign-stage refusal). Human review pending on all; sirosfoundation/go-wallet-backend#470 is not included. Rollback pins are kept in the comments.Config in this branch (from #79 and #80)
verifier.nym_auth(allowlist from zk-cred-longfellowprofile.rs, artifact baked in bydockerfiles/verifier-nymauth.Dockerfile),preferred_vp_formats.zkp_req,verifier_id=x509_san_dns:sirosid-gdc-vc-verifier.fly.dev(scopes every pseudonym; changing it changes every pseudonym), presetsnym_auth_pseudonymandnym_auth_pseudonym_age, and the opt-insigningCertDnsSan(re-issues the signing leaf from the same key and root). No engine config key changed.Verified live on gdc after the verifier deploy
Startup clean; GET
/in a headless browser lists both presets (the page was a spinner with the previous image);POST /ui/interactionwith a preset-derived query yields a request object withclient_idx509_san_dns:sirosid-gdc-vc-verifier.fly.dev,response_modedirect_post.jwt, an x5c leaf with the dNSNamesirosid-gdc-vc-verifier.fly.dev, and a dcql credential of formatzkp_reqwithmeta {doctype_value, source_format: mso_mdoc}and_reqcarrying the pseudonym (and, for the age preset,age_check) items. Issuer/.well-knownis advertisement-only. Three Pixel issuance rounds (seed delivered,/notification204, engine forwarded,/v1/evaluatetrue) ran on earlier pins. A real wallet presentation is expected to fail until the proof's verification time T is agreed between wallet and verifier.PDP verified live (2026-10-21, after the 0.26.0 deploy)
Replays against the live PDP: verifier path (
x509_san_dns:sirosid-gdc-vc-verifier.fly.dev, the gdc x5c chain) forcredential-verifierandverifierreturn decision true from the Fly whitelist andcontext.verifier_ids=[{x509_san_dns, sirosid-gdc-vc-verifier.fly.dev, basis client-id-binding}]; issuermdoc-issuerandmdoc-issuer-auth(mDOC IACA registry) andcredential-issuer(whitelistpid-issuers) are true with noverifier_ids, identical to 0.25.0 (also compared locally against 0.25.0 with the same rendered config); a self-signed chain with the same dNSName is false.Not done here
WMP discovery on gdc (
engine_url, the proxy route for/.well-known/wmp-configuration,proxy_buffering offfor SSE) is a follow-up for when an SDK uses WMP.🤖 Generated with Claude Code