Skip to content

Feature/175/lambda request id in logs minimal#204

Merged
pjhartzell merged 3 commits intostac-utils:mainfrom
pjhartzell:feature/175/lambda-request-id-in-logs-minimal
Jan 14, 2026
Merged

Feature/175/lambda request id in logs minimal#204
pjhartzell merged 3 commits intostac-utils:mainfrom
pjhartzell:feature/175/lambda-request-id-in-logs-minimal

Conversation

@pjhartzell
Copy link
Copy Markdown
Collaborator

@pjhartzell pjhartzell commented Jan 7, 2026

Related Issue(s):

Proposed Changes:

  1. The self.logger creation in the Task initializer now picks up the aws_request_id kwarg and combines it with the payload id to generate the prefix for log messages.
  2. The logging message format is augmented with the AWS lambda Request ID, if it is passed:
    • It used to be [<payload-id>] <message>, which produced logging output like INFO:logger-name:[payload-id] Some message.
    • It can now be [<payload-id> <lambda-request-id] <message>, which produces logging output like INFO:logger-name:[payload-id lambda-request-id] Some message.
  3. The docs example now shows how to pass a Lambda request ID to the handler when building a stac-task for a Lambda.

Note: This is intentionally minimal. A deeper look at how stac-task handles logging is in progress.

PR Checklist:

  • I have added my changes to the CHANGELOG or a CHANGELOG entry is not required.

@pjhartzell pjhartzell requested a review from jkeifer January 8, 2026 14:16
@pjhartzell pjhartzell force-pushed the feature/175/lambda-request-id-in-logs-minimal branch from 3effb9e to 78b3c8e Compare January 14, 2026 19:17
@pjhartzell pjhartzell requested a review from jkeifer January 14, 2026 19:18
@pjhartzell pjhartzell force-pushed the feature/175/lambda-request-id-in-logs-minimal branch from 78b3c8e to 1085163 Compare January 14, 2026 19:25
@pjhartzell pjhartzell merged commit 1f57943 into stac-utils:main Jan 14, 2026
7 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants