feat(ocsf): carry structured events on sandbox log stream - #3104
feat(ocsf): carry structured events on sandbox log stream#3104krishicks wants to merge 1 commit into
Conversation
Review findings
|
Is this really the case? I know previously the supervisor was always using the latest image, but do we make any guarantees about using a newer supervisor with an older gateway? |
I think this should be done as a separate PR. Thoughts? |
I don't think we explicitly prevent this case. Maybe for pre-0.1.0 this doesn't matter. But I think the review agent was orienting around HA rolling upgrades where the mismatch could happen. |
4f2efeb to
362c556
Compare
Carry each sandbox OCSF event as JSON alongside the log line so the gateway and downstream exporters retain the complete event instead of reparsing shorthand text. Render display text from the decoded event at the gateway. Preserve the original message when a line has no structured payload or the payload cannot be decoded. Keep shorthand in the message field as a fallback for mixed-version gateways until October 15, 2026. Refs #1055 Signed-off-by: Kris Hicks <khicks@nvidia.com>
362c556 to
6933e73
Compare
Summary
Carry each sandbox OCSF event as JSON alongside the log line so the gateway and downstream exporters retain the complete event instead of reparsing shorthand text.
Render display text from the decoded event at the gateway. Preserve the original message when a line has no structured payload or the payload cannot be decoded.
Related Issue
#1055
Changes
Testing
mise run pre-commitpassesChecklist