Skip to content

Conversation

@adela-bytebase
Copy link
Contributor

No description provided.

Copilot AI review requested due to automatic review settings December 14, 2025 21:47
@vercel
Copy link

vercel bot commented Dec 14, 2025

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Review Updated (UTC)
bytebase-com Ready Ready Preview, Comment Dec 18, 2025 9:24am

Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR enhances the audit log documentation by expanding the section on streaming audit logs to external systems. The update transforms a brief explanation into a comprehensive guide with configuration steps, output format examples, integration guidance, and supported systems.

Key Changes

  • Expanded the audit log streaming section with structured subsections for configuration, output formats, integration, and supported systems
  • Added detailed examples showing both key-value pair and JSON output formats
  • Included Docker command example demonstrating the --enable-json-logging flag usage

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.


Database permission controls individual users' or groups' actions within the database. Below shows the built-in roles' database permissions.

| Role | EXPLAIN | Query | Export | Mutation DML | DDL | Admin |
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why not simply DML?
DML = UPDATE | INSERT | DELETE.
What does the word mutation add?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

there page is not updated, it's because the folder name mintlify->docs
I will rename this part in another pr

![filter-date](/content/docs/security/audit-log/filter-date.webp)

## Emit audit logs to stdout
## Stream audit logs to external systems
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Let's take this chance to do a major refinement of this doc, as audit log is really important for our customers in terms of compliance requirement, e.g. Singpass requires comprehensive audit logging to integrate into their system for monitoring purposes.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Existing content, especially the Streaming part are mostly good. The whole structure needs some improvement.
Docs are mostly used by developers to solve technical problems, so our doc should be structured to help them quickly obtain the necessary information.
They usually use audit log doc to:

  1. integrate our audit log into their system
  2. find the culprit of incident
    both of the use cases need log structure and what events are logged to accomplish this

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Actually the structure and documentation of https://tailscale.com/kb/1203/audit-logging is a very good example, we can follow the pattern:

  • Basic Information:
    • Introducing what audit logging do, and give a reference to all events (log events can be placed in the end of the doc in reference section.
    • Available only for Pro and Enterprise
    • Log structure.
  • Accessing Audit Logs
    • on UI
    • via API
    • Streaming (it would be nice if we include example on end-to-end configuration)
  • Retention: need to warn user to purge periodically or disk may be full and result in Bytebase crash
  • Events
  • Limitations:
    • events we don't log
    • query results won't be included in the log due to privacy and security reason


The stdout logs can be ingested by any log collection system, including:

- **SIEM platforms**: Splunk, Datadog, Elastic, Sumo Logic, Panther
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Future improvement, add example to integrate log streaming to:

  1. splunk: dominant SIEM for enterprises
  2. elasticsearch with logstash: mostly used in open source community
    @zchpeter

@adela-bytebase adela-bytebase merged commit b4f3b9d into main Dec 24, 2025
4 of 5 checks passed
@adela-bytebase adela-bytebase deleted the a-branch-95 branch December 24, 2025 02:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants