Skip to content

clarify trust_anchor - #52

Merged
zachmann merged 7 commits into
mainfrom
gz-require-ta-parameter
Oct 8, 2026
Merged

zachmann merged 7 commits into
mainfrom
gz-require-ta-parameter

Conversation

@zachmann

Copy link
Copy Markdown
Member

Resolves #36

Originally I thought it would be convenient for clients to have the trust_anchor request parameter optional, since I would expect that it is likely to use the same entity that provides the entity collection endpoint as the trust anchor.

When thinking about how to clarify #36 I cam to the conclusion that it is probably the best to instead require the trust_anchor parameter. This makes usage more clear for implementers.

Comment thread openid-federation-entity-collection-1_0.md
@zachmann
zachmann force-pushed the gz-require-ta-parameter branch from c2c3fb9 to 2ec5860 Compare July 13, 2026 08:53
@zachmann zachmann changed the title make trust_anchor parameter required clarify trust_anchor Jul 13, 2026
@zachmann
zachmann requested a review from lj-raidiam July 13, 2026 08:54
@zachmann
zachmann requested a review from selfissued October 7, 2026 13:59
Comment thread openid-federation-entity-collection-1_0.md Outdated
Comment thread openid-federation-entity-collection-1_0.md Outdated

@selfissued selfissued left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please apply my suggestions before merging

Co-authored-by: Michael B. Jones <michael_b_jones@hotmail.com>
@zachmann
zachmann deployed to github-pages October 8, 2026 04:57 — with GitHub Actions Active
Co-authored-by: Michael B. Jones <michael_b_jones@hotmail.com>
@zachmann
zachmann deployed to github-pages October 8, 2026 04:57 — with GitHub Actions Active
@zachmann
zachmann merged commit 3cde4f6 into main Oct 8, 2026
1 check passed
@zachmann
zachmann deleted the gz-require-ta-parameter branch October 8, 2026 05:09

This branch was successfully deployed

1 active deployment
github-pages — ed726b3a Deployed Oct 8, 2026 by zachmann via build-and-deploy #72
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.

Confusing description of trust_anchor parameter

3 participants