Skip to content

Conversation

mgoworko
Copy link
Collaborator

@mgoworko mgoworko commented Jul 25, 2025

🚀New (ES) added new audit serializers - ReportingAllEventsAuditLogSerializer, ReportingAllEventsWithQueryAuditLogSerializer
🚀New (ES) There is an option to define custom audit serializer in config, without implementation
🐞Fix (ES) Restored backwards compatibility of custom audit log serializer implementations, that extend the DefaultAuditLogSerializer class. The custom serializers compiled against ROR 1.65 and 1.66, that use DefaultAuditLogSerializer have to be recompiled in order to work.

Summary by CodeRabbit

  • New Features
    • Added configurable audit serializer, verbosity-aware full-event serializers, and a reusable audit serialization helper with flexible field descriptors.
  • Changes
    • Audit config now supports structured static/configurable serializers; environment context is provided at audit-time; simpler serializer constructors and defaults.
  • Tests
    • Expanded integration and unit tests covering new serializers, configurable fields, and environment-aware behavior.
  • Removed
    • Legacy serializer implementations removed.
  • Chores
    • Build plugin version bumped and audit module test support added.

@mgoworko mgoworko marked this pull request as ready for review July 25, 2025 19:28

This comment was marked as outdated.

coderabbitai[bot]

This comment was marked as outdated.

@mgoworko mgoworko marked this pull request as draft July 25, 2025 20:16
@mgoworko mgoworko marked this pull request as ready for review July 28, 2025 15:35
@mgoworko mgoworko requested a review from coutoPL July 28, 2025 15:35
coderabbitai[bot]

This comment was marked as outdated.

coderabbitai[bot]

This comment was marked as outdated.

coderabbitai[bot]

This comment was marked as outdated.

coderabbitai[bot]

This comment was marked as outdated.

coderabbitai[bot]

This comment was marked as outdated.

coderabbitai[bot]

This comment was marked as outdated.

@mgoworko mgoworko requested a review from coutoPL August 29, 2025 12:47
coderabbitai[bot]

This comment was marked as resolved.

coderabbitai[bot]

This comment was marked as resolved.

@mgoworko mgoworko requested a review from coutoPL September 5, 2025 16:37
coderabbitai[bot]

This comment was marked as outdated.

@mgoworko mgoworko requested a review from coutoPL September 14, 2025 11:03
@mgoworko mgoworko requested a review from coutoPL September 14, 2025 17:51
coderabbitai[bot]

This comment was marked as outdated.

Copy link
Collaborator

@coutoPL coutoPL left a comment

Choose a reason for hiding this comment

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

LGTM.
Please don't forget about docs update.

coderabbitai[bot]

This comment was marked as resolved.

Copy link
Collaborator

@coutoPL coutoPL left a comment

Choose a reason for hiding this comment

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

LGTM

@mgoworko mgoworko merged commit f4fd3c4 into sscarduzio:develop Sep 16, 2025
21 checks passed
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.

2 participants