Conversation
|
@alexluong is attempting to deploy a commit to the Hookdeck team on Vercel, but is not a member of this team. To resolve this issue, you can:
To read more about collaboration on Vercel, click here. |
|
We'll need audit logs for tenants CRUD & destinations CRUD |
I think it's probably better to use the logger in |
|
Sure ultimately I just think the audit traces need to represent those actions |
|
@alexbouchardd for updates, do you think it's important to capture the data change, or would logging the updated destination JSON a good enough solution? For actions like "disable destination", is it better to log the specific "disable", or can we treat it as "update destination"? |
Here are all the audit logs we have in this PR:
You can look for them using
logger.Auditas a search term too.Questions:
1: One question I have is whether we should include the
callerfield in the log JSON. Caller is the file and line number that the log is called from.