search

oauth-wg / oauth-transaction-tokens

MIT License
7 stars 10 forks source link

Update draft-ietf-oauth-transaction-tokens.md #89

Closed gffletch closed 1 month ago

gffletch commented 3 months ago

Addresses text regarding the aud claim throughout the document. Issue #76

tulshi commented 3 months ago

Although your changes sufficiently capture what is needed, what I liked about the previous draft is that it clearly identified that each Trust Domain MUST have a unique identifier, which is set as the aud value. I wonder if we can say that somewhere (outside of the Terminology section)

gffletch commented 2 months ago

Ok, let me see if there is a good place to add that. I wonder if it will be easier to merge PR #90 first and then let me fix this one as there may be conflicts?

tulshi commented 2 months ago

Makes sense to work on this after we merge #90

On Fri, Apr 26, 2024 at 3:11 PM George Fletcher @.***> wrote:

Ok, let me see if there is a good place to add that. I wonder if it will be easier to merge PR #90 https://github.com/oauth-wg/oauth-transaction-tokens/pull/90 first and then let me fix this one as there may be conflicts?

— Reply to this email directly, view it on GitHub https://github.com/oauth-wg/oauth-transaction-tokens/pull/89#issuecomment-2080179762, or unsubscribe https://github.com/notifications/unsubscribe-auth/AB55UG77Y2GZKFE7FSVPC7LY7LGIFAVCNFSM6AAAAABFO22ZLKVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDAOBQGE3TSNZWGI . You are receiving this because your review was requested.Message ID: @.***>