docs: document Tempo session v2 protocol#278
Conversation
Spec Preview
|
![chatgpt-codex-connector[bot]](https://avatars.githubusercontent.com/in/1144995?s=60&v=4){kind=small}
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 6c197388d7
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
| "escrowContract": "0x9d136eEa063eDE5418A6BC7bEafF009bBb6CFa70", | ||
| "chainId": 42431 | ||
| "chainId": 42431, | ||
| "sessionProtocol": "v2" |
There was a problem hiding this comment.
Setting this challenge to sessionProtocol: "v2" makes the rest of the complete example non-compliant: the immediately following open, voucher, and close payloads omit descriptor, even though this commit makes descriptor REQUIRED for v2 credentials. Implementations copying this worked example would send credentials that compliant v2 servers must reject; either add the descriptor to those payloads or leave this end-to-end example as v1.
Useful? React with 👍 / 👎.
brendanjryan
force-pushed
the
brendanjryan/tempo-session-v2-spec
branch
from
6c19738 to
7a7aec3
Compare
1 participant
Summary