Skip to content

[v17] Fix large event handling for Athena backend#63551

Merged
kshi36 merged 1 commit intobranch/v17from
kevin/backport-63522-branch/v17
Feb 6, 2026
Merged

[v17] Fix large event handling for Athena backend#63551
kshi36 merged 1 commit intobranch/v17from
kevin/backport-63522-branch/v17

Conversation

@kshi36
Copy link
Copy Markdown
Contributor

@kshi36 kshi36 commented Feb 5, 2026

Backport #63522 to branch/v17

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/v17 is apievents.AuditEvent instead of events.EventFields.

@kshi36 kshi36 force-pushed the kevin/backport-63522-branch/v17 branch from 734c370 to dd84acb Compare February 5, 2026 20:22
@kshi36 kshi36 marked this pull request as ready for review February 5, 2026 21:10
@kshi36 kshi36 requested review from hugoShaka and tigrato February 5, 2026 21:10
@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 6, 2026
Merged via the queue into branch/v17 with commit 5e12202 Feb 6, 2026
41 checks passed
@kshi36 kshi36 deleted the kevin/backport-63522-branch/v17 branch February 6, 2026 18:43
This was referenced 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