cdevents / spec

A common specification for Continuous Delivery events
Apache License 2.0
129 stars 22 forks source link

inconsistent or confusing cloudevent type mapping in cloudevents-binding spec #178

Closed strangemonad closed 8 months ago

strangemonad commented 9 months ago

The cloudevents-binding spec states that the cloudevent type must be set to the cdevent type but then the included HTTP example has a mismatch between the ce-type: dev.cdevents.taskrun.started.0.1-draft header and the data's "context"."type":"dev.cdevents.taskrun.started" field.

afrittoli commented 8 months ago

Thanks @strangemonad for reporting this, that is indeed an oversight, I will make a PR to fix it.