Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update CAEP & RISC Events and SSF Docs with txn claim #157

Open
iamseanodentity opened this issue Apr 24, 2024 · 2 comments
Open

Update CAEP & RISC Events and SSF Docs with txn claim #157

iamseanodentity opened this issue Apr 24, 2024 · 2 comments
Assignees
Labels
v3 Address this issue before the v3 cutoff (June 15, 2024)

Comments

@iamseanodentity
Copy link

The txn claim is called out as optional on RFC 8417 but is not referenced on CAEP / RISC Events or in the general SSF documentation. With the new CAEP Event being introduced, session established, it feels like introducing txn now is the right time before v1 is established. This speaks to how a transmitter(SST) and receiver(SSR) can co-op using a standard JWT claim.

SST -> session revoked CAEP Event -> SSR with txn: 123. The SSR that received the SET can then send back, acting as a SST, a session revoked event to the Transmitter which is now acting as a SSR. This is a good example of auditing and accounting practices. This also helps inform a SST that the signals it is emititng to the SSR's are still accurate and not subjective noise to a SSR (i.e. the underlying data powering the SST is valid and accurate...which is important).

Would like to update the open-id-caep-spec (Sections 2 and 3), open-id-risc-spec (Sections 2 and 3) and open-id-sharedsignals-framework-1.0 (Section 5) md files. In the events I think we would want to call out the specifics to use the txn claim and not have it optional for all events.

@FragLegs
Copy link
Contributor

For clarity's sake, the proposal here is to:

  1. Update the SSF spec to make txn a required top-level claim of the SET for all SSF events (top-level meaning it nests at the same level as iss and aud in the SET json).
  2. Update the SSF spec to include txn in any non-normative examples of events.
  3. Update the CAEP and RISC specs to include txn in the non-normative examples.

Is that correct?

@FragLegs FragLegs added the v3 Address this issue before the v3 cutoff (June 15, 2024) label May 16, 2024
@iamseanodentity
Copy link
Author

PR submitted

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
v3 Address this issue before the v3 cutoff (June 15, 2024)
Projects
None yet
Development

No branches or pull requests

2 participants