Skip to content

feat: add request id headers to aws lambda start/end invocation calls #4136

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
merged 3 commits into from
Nov 18, 2025
Merged

feat: add request id headers to aws lambda start/end invocation calls #4136

merged 3 commits into from
Nov 18, 2025
+98 −1

Conversation

@ojproductions
Copy link
Contributor

@ojproductions ojproductions commented Nov 18, 2025 •
edited by atlassian bot
Loading

This change was copied over from this corresponding datadog-lambda-go pr

What does this PR do?

Adds the lambda request Id as headers to the start/end-invocation http calls

Motivation

APMSVLS-273

Testing Guidelines

  • Added unit tests
  • ran the extension and printed the header contents as shown below:
headers logs

Additional Notes

Types of changes

  • Bug fix
  • New feature
  • Breaking change
  • Misc (docs, refactoring, dependency upgrade, etc.)

Checklist

  • This PR's description is comprehensive
  • This PR contains breaking changes that are documented in the description
  • This PR introduces new APIs or parameters that are documented and unlikely to change in the foreseeable future
  • This PR impacts documentation, and it has been updated (or a ticket has been logged)
  • This PR's changes are covered by the automated tests
  • This PR collects user input/sensitive content into Datadog

@ojproductions ojproductions requested review from a team as code owners November 18, 2025 14:33
@github-actions github-actions bot added the apm:ecosystem contrib/* related feature requests or bugs label Nov 18, 2025
@pr-commenter
Copy link

pr-commenter bot commented Nov 18, 2025 •
edited
Loading

Benchmarks

Benchmark execution time: 2025-11-18 18:38:39

Comparing candidate commit 046b192 in PR branch onzia/aws-request-id-header with baseline commit 7ce9243 in branch main.

Found 0 performance improvements and 0 performance regressions! Performance is the same for 24 metrics, 0 unstable metrics.

@ojproductions ojproductions mentioned this pull request Nov 18, 2025
Merged
10 tasks
@ojproductions ojproductions changed the title add request id headers to aws lambda start/end invocation calls feat: add request id headers to aws lambda start/end invocation calls Nov 18, 2025
@datadog-official

This comment has been minimized.

Sign in to view
@ojproductions
Copy link
Contributor Author

ojproductions commented Nov 18, 2025

/merge

@dd-devflow-routing-codex
Copy link

dd-devflow-routing-codex bot commented Nov 18, 2025 •
edited
Loading

View all feedbacks in Devflow UI.

2025-11-18 19:12:11 UTC ℹ️ Start processing command /merge

2025-11-18 19:12:16 UTC ℹ️ MergeQueue: pull request added to the queue

The expected merge time in main is approximately 31m (p90).

2025-11-18 19:58:40 UTC ℹ️ MergeQueue: This merge request was merged

@dd-mergequeue dd-mergequeue bot merged commit 1afb00d into main Nov 18, 2025
336 of 339 checks passed
@dd-mergequeue dd-mergequeue bot deleted the onzia/aws-request-id-header branch November 18, 2025 19:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@purple4reina purple4reina purple4reina approved these changes

@duncanista duncanista duncanista approved these changes

Assignees

@purple4reina purple4reina

Labels

apm:ecosystem contrib/* related feature requests or bugs mergequeue-status: done

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@ojproductions @purple4reina @duncanista