search

xmidt-org / wrp-go

go implementation of the Web Routing Protocol
Apache License 2.0
4 stars 7 forks source link

Add UUID as a part of SimpleEvent #61

Open kristinapathak opened 3 years ago

kristinapathak commented 3 years ago

Currently, glaukos is utilizing a bug in Caduceus in order to skip over an event in the event history that is the same event sent to the webhook. The transaction UUID is meant to be no longer be set/created in Caduceus as a part of this Caduceus issue: https://github.com/xmidt-org/caduceus/issues/174#issuecomment-665274785

A UUID is helpful for identifying events. It would be nice to add it as a required field to SimpleEvent in some form, even if TransactionUUID doesn't make sense.

schmidtw commented 3 years ago

I agree, I think excluding the TransactionUUID was a mistake in the SimpleEvent. We'll need to rev the spec to add this.

kristinapathak commented 3 years ago

This would implement the change in the spec described in https://github.com/xmidt-org/docs/issues/74.