Skip to content

Commit

Permalink
Merge #108623
Browse files Browse the repository at this point in the history
108623: docgen: Restrict SQL activity logging by user/role doc update r=nickvigilante a=nickvigilante

Fixes DOC-8560

Release note: None

Co-authored-by: Nick Vigilante <[email protected]>
  • Loading branch information
craig[bot] and Nick Vigilante committed Oct 6, 2023
2 parents 36601cc + ea193ed commit c5474dd
Show file tree
Hide file tree
Showing 4 changed files with 50 additions and 0 deletions.
2 changes: 2 additions & 0 deletions docs/generated/logging.md
Original file line number Diff line number Diff line change
Expand Up @@ -129,6 +129,8 @@ data access to sensitive data:

- Data access audit events (when table audit is enabled via
[ALTER TABLE ... EXPERIMENTAL_AUDIT](alter-table.html#experimental_audit))
- Data access audit events (when role-based audit is enabled via
[`sql.log.user_audit` cluster setting](role-based-audit-logging.html#syntax-of-audit-settings))
- SQL statements executed by users with the admin role
- Operations that write to system tables

Expand Down
2 changes: 2 additions & 0 deletions pkg/util/log/channel/channel_generated.go

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

44 changes: 44 additions & 0 deletions pkg/util/log/log_channels_generated.go

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

2 changes: 2 additions & 0 deletions pkg/util/log/logpb/log.proto
Original file line number Diff line number Diff line change
Expand Up @@ -148,6 +148,8 @@ enum Channel {
//
// - Data access audit events (when table audit is enabled via
// [ALTER TABLE ... EXPERIMENTAL_AUDIT](alter-table.html#experimental_audit))
// - Data access audit events (when role-based audit is enabled via
// [`sql.log.user_audit` cluster setting](role-based-audit-logging.html#syntax-of-audit-settings))
// - SQL statements executed by users with the admin role
// - Operations that write to system tables
//
Expand Down

0 comments on commit c5474dd

Please sign in to comment.