Skip to content

[plugins] add plugin audit events boilerplate to CUD operations#45161

Merged
tigrato merged 2 commits intomasterfrom
tigrato/add-plugins-audit-logs
Aug 8, 2024
Merged

[plugins] add plugin audit events boilerplate to CUD operations#45161
tigrato merged 2 commits intomasterfrom
tigrato/add-plugins-audit-logs

Conversation

@tigrato
Copy link
Copy Markdown
Contributor

@tigrato tigrato commented Aug 6, 2024

This PR adds the required boilerplate code to emit audit events for create, update and delete operations.

This PR adds the required boilerplate code to emit audit events for `create`, `update` and `delete` operations.

Signed-off-by: Tiago Silva <tiago.silva@goteleport.com>
@tigrato tigrato added backport/branch/v14 no-changelog Indicates that a PR does not require a changelog entry labels Aug 6, 2024
@github-actions github-actions Bot added audit-log Issues related to Teleports Audit Log size/md labels Aug 6, 2024
@github-actions github-actions Bot requested review from AntonAM and codingllama August 6, 2024 18:36
Comment thread api/proto/teleport/legacy/types/events/events.proto Outdated
Comment thread api/proto/teleport/legacy/types/events/events.proto Outdated
@tigrato tigrato enabled auto-merge August 8, 2024 09:33
@tigrato tigrato added this pull request to the merge queue Aug 8, 2024
Merged via the queue into master with commit 433febc Aug 8, 2024
@tigrato tigrato deleted the tigrato/add-plugins-audit-logs branch August 8, 2024 10:07
@public-teleport-github-review-bot
Copy link
Copy Markdown

@tigrato See the table below for backport results.

Branch Result
branch/v14 Failed
branch/v15 Failed
branch/v16 Failed

tigrato added a commit that referenced this pull request Sep 11, 2024
* [plugins] add plugin audit events boilerplate to CUD operations

This PR adds the required boilerplate code to emit audit events for `create`, `update` and `delete` operations.

Signed-off-by: Tiago Silva <tiago.silva@goteleport.com>

* handle comments

---------

Signed-off-by: Tiago Silva <tiago.silva@goteleport.com>
github-merge-queue Bot pushed a commit that referenced this pull request Sep 11, 2024
…) (#46497)

* [plugins] add plugin audit events boilerplate to CUD operations

This PR adds the required boilerplate code to emit audit events for `create`, `update` and `delete` operations.



* handle comments

---------

Signed-off-by: Tiago Silva <tiago.silva@goteleport.com>
tigrato added a commit that referenced this pull request Sep 21, 2024
* [plugins] add plugin audit events boilerplate to CUD operations

This PR adds the required boilerplate code to emit audit events for `create`, `update` and `delete` operations.

Signed-off-by: Tiago Silva <tiago.silva@goteleport.com>

* handle comments

---------

Signed-off-by: Tiago Silva <tiago.silva@goteleport.com>
tigrato added a commit that referenced this pull request Sep 21, 2024
* [plugins] add plugin audit events boilerplate to CUD operations

This PR adds the required boilerplate code to emit audit events for `create`, `update` and `delete` operations.

Signed-off-by: Tiago Silva <tiago.silva@goteleport.com>

* handle comments

---------

Signed-off-by: Tiago Silva <tiago.silva@goteleport.com>
github-merge-queue Bot pushed a commit that referenced this pull request Oct 3, 2024
…#46838)

* [plugins] add plugin audit events boilerplate to CUD operations (#45161)

* [plugins] add plugin audit events boilerplate to CUD operations

This PR adds the required boilerplate code to emit audit events for `create`, `update` and `delete` operations.

Signed-off-by: Tiago Silva <tiago.silva@goteleport.com>

* handle comments

---------

Signed-off-by: Tiago Silva <tiago.silva@goteleport.com>

* drop `types.PluginsV1` from audit log payload (#46830)

Okta plugin sets `plugin.spec.settings` which is a one of message and our audit log incorrectly uses `json.Marshal/Unmarshal` to marshal and Unmarshal objects from json.

When unmarshaling the oneof, json parser fails because it can't decode the correct message.

This PR changes the type to use a `google.protobuf.Struct` so it can hold any static data.

Signed-off-by: Tiago Silva <tiago.silva@goteleport.com>

* add TimToMaxSize

---------

Signed-off-by: Tiago Silva <tiago.silva@goteleport.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

audit-log Issues related to Teleports Audit Log no-changelog Indicates that a PR does not require a changelog entry size/md

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants