Skip to content

[v18] Fix large event handling for Athena backend#63550

Merged
kshi36 merged 1 commit intobranch/v18from
kevin/backport-63522-branch/v18
Feb 5, 2026
Merged

[v18] Fix large event handling for Athena backend#63550
kshi36 merged 1 commit intobranch/v18from
kevin/backport-63522-branch/v18

Conversation

@kshi36
Copy link
Copy Markdown
Contributor

@kshi36 kshi36 commented Feb 5, 2026

Backport #63522 to branch/v18

changelog: Fixed bug where event handler would throw an error on Athena backend when handling large events

// Exact rb.totalSize doesn't really matter since the response is
// already size limited.
rb.totalSize += events.MaxEventBytesInResponse
rb.output = append(rb.output, event)
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Manual backport because return type for branch/v18 is apievents.AuditEvent instead of events.EventFields.

@kshi36 kshi36 marked this pull request as ready for review February 5, 2026 20:23
@kshi36 kshi36 requested review from hugoShaka and tigrato February 5, 2026 20:23
@github-actions github-actions bot added audit-log Issues related to Teleports Audit Log backport size/sm labels Feb 5, 2026
@kshi36 kshi36 added this pull request to the merge queue Feb 5, 2026
Merged via the queue into branch/v18 with commit 6501ba3 Feb 5, 2026
44 checks passed
@kshi36 kshi36 deleted the kevin/backport-63522-branch/v18 branch February 5, 2026 21:30
@doggydogworld doggydogworld mentioned this pull request Feb 9, 2026
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 backport size/sm

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants