cirruslabs / cirrus-ci-docs

Documentation for Cirrus CI 📚
https://cirrus-ci.org
MIT License
349 stars 109 forks source link

Document upcoming changes to webhooks payloads #1282

Open fkorotkov opened 2 months ago

fkorotkov commented 2 months ago

These are proposed new fields that will come to the payloads and GraphQL API in the coming weeks.

These events will allow to improve export of the events into systems like Datadog. See the following issue:

fkorotkov commented 2 months ago

@edigaryev what do you think about a request header. Since the payload is a GraphQL query response and I don't think we can add timestamp there. 🤔

edigaryev commented 2 months ago

@edigaryev what do you think about a request header. Since the payload is a GraphQL query response and I don't think we can add timestamp there. 🤔

That would totally work too.

edigaryev commented 2 months ago

Since the payload is a GraphQL query response and I don't think we can add timestamp there. 🤔

On a side note, it seems that the action and old_status fields are merged somehow, so probably we can:

Screenshot 2024-07-22 at 15 49 20
edigaryev commented 2 months ago

Also, perhaps we should make the audit_event to conform to this format, or vice versa, because currently it differs, see screenshot below.

Note that the body is not a nested object, but instead a JSON string:

Screenshot 2024-07-22 at 15 57 21