sharedsignals icon indicating copy to clipboard operation
sharedsignals copied to clipboard

Fixes #221 - Add event_metadata to CAEP and RISC

Open jischr opened this issue 9 months ago • 2 comments

Added event_metadata to CAEP and RISC for the transmitter to provide additional information about the event itself.

Fixes #221

jischr avatar Mar 31 '25 23:03 jischr

@appsdesh

  • what does it mean by action?
  • that wording was taken from the issue. happy to remove or replace if it doesnt suit.
  • how would we prevent bloating?
  • I don't believe we need to prevent bloating as SETs do not have restrictions in place for event claims. if necessary, we could put restrictions on string length.
  • Why are we singling this out "This claim SHOULD not be used to express information about the subject." needs more concrete guidance
  • We need to differentiate the this claim for other claims that could have been added to the individual event schemas (that describes the subject). this is metadata. Guidance could be added to prevent this claim from becoming out of hand.

jischr avatar Apr 01 '25 01:04 jischr

"action" here is needed, but we do not need to call it out in the description. In the event_metadata field it is setting the foundation to expand upon profile events that might be mandatory via a contract between the transmitter and receiver.

We need guidelines for this and a use case with event_metadata details.

  • Don't overload this claim (size considerations of the JWT)
  • Don't add in subject based information and keep it siloed to the event itself (i.e. correlation_id, enriched meta data info)
  • Why and When to use it

iamseanodentity avatar Apr 01 '25 18:04 iamseanodentity

@iamseanodentity do you can an example of this you can supply?

jischr avatar Apr 15 '25 13:04 jischr

@iamseanodentity do you can an example of this you can supply?

I will get to this tonight. Would you like me to add inline or just here and you can handle the copy?

iamseanodentity avatar Apr 15 '25 15:04 iamseanodentity

decided against this 4/15

jischr avatar Apr 22 '25 17:04 jischr