Skip to content

fix(apache): add custom.date as fallback source for ocsf.time#22986

Open
jbfeldman-dd wants to merge 1 commit intomasterfrom
fix/ocsf-apache-time-mapping
Open

fix(apache): add custom.date as fallback source for ocsf.time#22986
jbfeldman-dd wants to merge 1 commit intomasterfrom
fix/ocsf-apache-time-mapping

Conversation

@jbfeldman-dd
Copy link
Contributor

Summary

  • The Apache OCSF sub-pipeline maps date_accessocsf.time, but when the Datadog Agent pre-parses Apache access logs it stores the timestamp as custom.date instead of date_access
  • The remapper found no value for date_access in these pre-parsed events, leaving ocsf.time unset
  • This caused ~2,100 attribute_required_missing validation errors per day (time missing) in the OCSF validation job

Change: Add custom.date as a second source on the existing ocsf.time schema-remapper. The date_access source remains first so Agent-side-parsed events that do have it are unaffected.

Test plan

  • Verify that pre-parsed Apache logs (with custom.date) have ocsf.time populated after the pipeline runs
  • Confirm attribute_required_missing error count drops for source apache in event-jobs-security-monitoring
  • Confirm logs that were already passing (with date_access) continue to pass

🤖 Generated with Claude Code

When the Datadog Agent pre-parses Apache access logs, it stores the
timestamp as custom.date (epoch ms) rather than date_access. The OCSF
sub-pipeline's time remapper only looked at date_access, so ocsf.time
was missing for these events, causing ~2,100 attribute_required_missing
validation errors per day.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@github-actions
Copy link
Contributor

⚠️ Recommendation: Add qa/skip-qa label

This PR does not modify any files shipped with the agent.

To help streamline the release process, please consider adding the qa/skip-qa label if these changes do not require QA testing.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant